Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Domain: Session service #42

Open
lhaze opened this issue Jan 15, 2019 · 1 comment
Open

Domain: Session service #42

lhaze opened this issue Jan 15, 2019 · 1 comment

Comments

@lhaze
Copy link
Collaborator

lhaze commented Jan 15, 2019

  • ISession interface
  • Session API:
    • session expiration (set_expiry/get_expiry/flush)
    • session as data storage with dict API
    • information about the user (ISessionWithUser)
@lhaze
Copy link
Collaborator Author

lhaze commented Jan 18, 2019

One clarification: there's no need for making session model/service/whatever user-model-aware. Two reasons:

  • There are lots of applications where having a session instance doesn't imply having a user model instance specified. Maybe there's no user model at all.
  • It seems quite opposite dependency. There is some kind of authentication service that depends on Session component (how about client-side "sessions", even if they are security anti-pattern?), that reads some value from the session and interprets it as an identifier for user repository to get an instance. This seems to be The Clean Architecture way.

@lhaze lhaze changed the title Domain: Session pattern Domain: Session service Mar 15, 2019
@lhaze lhaze modified the milestones: 0.2, 0.3 Dec 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant