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

Prevent server leak #126

Open
eric-burel opened this issue Jun 2, 2022 · 0 comments
Open

Prevent server leak #126

eric-burel opened this issue Jun 2, 2022 · 0 comments
Labels
easy / good first issue Good for newcomers enhancement New feature or request help wanted Extra attention is needed internship

Comments

@eric-burel
Copy link
Collaborator

eric-burel commented Jun 2, 2022

Is your feature request related to a problem? Please describe.
Currently: server folder or server.ts means server-only code => but we don't strictly enforce that so there's a risk of leak

Describe the solution you'd like
Prevent user from accidentally loading server-only code client-side

Describe alternatives you've considered

  • An eslint rule that detect that you are not in a "server" file (no server folder/no server.ts) but you import a "server" code accidentally
  • A webpack plugin that fails the client bundle if server code is detected
@eric-burel eric-burel added enhancement New feature or request easy / good first issue Good for newcomers help wanted Extra attention is needed internship labels Jun 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy / good first issue Good for newcomers enhancement New feature or request help wanted Extra attention is needed internship
Projects
None yet
Development

No branches or pull requests

1 participant