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

Export session to @lunoxjs/session #45

Merged
merged 10 commits into from
Jun 12, 2023
Merged

Export session to @lunoxjs/session #45

merged 10 commits into from
Jun 12, 2023

Conversation

axmad386
Copy link
Contributor

@axmad386 axmad386 commented Jun 12, 2023

What's Changes

  • feat(lunox-session): lunox session exported to @lunoxjs/session
  • feat(lunox-session): request macro session to get SessionManager
  • feat(lunox-session): VerifyCsrfToken middleware
  • style: format code
  • feat(lunox-api): remove session from dependecies
  • feat(lunox-react): change session dependencies to @lunoxjs/session
  • feat(lunox-svelte): change session dependencies to @lunoxjs/session
  • feat(lunox-api): remove @lunoxjs/auth since its depends on @lunoxjs/session
  • refactor(lunox-core): macroable now using Object.prototype instead of useMagic
  • chore: bump lunox packages version

Note:

  • @lunoxjs/core/contracts now not exporting type Request. Please use Request exported from @lunoxjs/core directly. Same happen to Router type.
    // before
    import type {Request, Router} from "@lunoxjs/core/contracts";
    // after
    import type {Request, Router} from "@lunoxjs/core";
  • @lunoxjs/auth now depend on @lunoxjs/session. May be we need to make abstraction to auth driver, so we can decouple them. For example when auth is using stateless like JWT.

@axmad386 axmad386 merged commit a5d35e2 into next Jun 12, 2023
@axmad386 axmad386 deleted the lunox-session branch June 12, 2023 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant