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

AuthenticateWithSessionCookieFailureReason isn't unavailable to import #1198

Open
monolithed opened this issue Jan 10, 2025 · 0 comments
Open

Comments

@monolithed
Copy link

monolithed commented Jan 10, 2025

I’m trying to avoid duplicating business logic and reducing the risk of errors in service responses by importing constants from AuthenticateWithSessionCookieFailureReason. However, I’ve run into two issues:

The file containing these constants, import {AuthenticateWithSessionCookieFailureReason} from '@workos-inc/node/lib/user-management/interfaces/authenticate-with-session-cookie.interface';, isn’t accessible for import.

My suggestion:
Properly export the constants from AuthenticateWithSessionCookieFailureReason for safer error handling and future package updates. A better approach would be to move these constants out of the interface file to a more accessible location since they aren’t really interfaces, and access to them is genuinely needed.

"next": "14.1.4", "@workos-inc/node": "7.36.0",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant