-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
NativeAuth Update #3146
base: minor
Are you sure you want to change the base?
NativeAuth Update #3146
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
22aaf62
to
118d28c
Compare
118d28c
to
8fe3a3b
Compare
8fe3a3b
to
4c6a397
Compare
d5a0ec4
to
1b2b095
Compare
1b2b095
to
54474a5
Compare
54474a5
to
9db7dfc
Compare
@@ -10,6 +10,9 @@ export default defineConfig({ | |||
* automatically fail for going over the 5 second default timeout. | |||
*/ | |||
testTimeout: process.env.E2E_DEBUG ? 1800 * 1000 : process.env.CI ? 30 * 1000 : 15 * 1000, | |||
sequence: { | |||
concurrent: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concurrency should be implicitly false otherwise tests are 100% failing
9db7dfc
to
9b8c892
Compare
9b8c892
to
8af73e9
Compare
8af73e9
to
b22e4b4
Compare
Hi, |
Description
Follow up PR for the #2786
Breaking changes
registerCustomerAccount
now returnsEmailAddressConflictError
if the customer has no user and no nativeAuth method. The method logic has been simplified with early termination. I removed the token refreshing logic from the first PR, as I believe it should not be the concern of the registration method.For enumeration attacks, several types of enumeration attacks are possible. Addressing them at the application-wide or software level may not be the best approach. Handling them at the load balancer level could be a more effective solution.
Checklist
📌 Always:
👍 Most of the time: