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

User API endpoint #18

Closed
Nininanou16 opened this issue Aug 14, 2023 · 5 comments
Closed

User API endpoint #18

Nininanou16 opened this issue Aug 14, 2023 · 5 comments
Labels
bug Something isn't working
Milestone

Comments

@Nininanou16
Copy link
Member

The accounts api endpoint throws an error, failing to acquire the req.session. Used to work, might have a look at the past edit for the checkJWT and authentication methods

@Nininanou16 Nininanou16 added the bug Something isn't working label Aug 14, 2023
@Nininanou16 Nininanou16 added this to the Beta 1.0 milestone Aug 14, 2023
@Nininanou16 Nininanou16 self-assigned this Aug 14, 2023
@Nininanou16 Nininanou16 moved this to Urgent in Solar Aug 14, 2023
@Nininanou16
Copy link
Member Author

caused whole auth issue when removing the commenting line 12 of checkJWT.ts, sending 401 status on every login request. To be explored, with error - uncaughtException: Error [ERR_STREAM_WRITE_AFTER_END]: write after end error in console. Might be caused due to the implementation of 2FA in login.tsx

@Nininanou16
Copy link
Member Author

caused whole auth issue when removing the commenting line 12 of checkJWT.ts, sending 401 status on every login request. To be explored, with error - uncaughtException: Error [ERR_STREAM_WRITE_AFTER_END]: write after end error in console. Might be caused due to the implementation of 2FA in login.tsx

Not experiencing this issue on accounts with 2FA disabled, seems to confirm this theory

@Nininanou16
Copy link
Member Author

Seems to be related to a larger auth issue. When reverting the temporary auth bypass in this commit.

@Nininanou16
Copy link
Member Author

Linked to #19 as a larger authentification issue, must be investigated in the upcoming days

@Nininanou16
Copy link
Member Author

Linked to #19 as a larger authentification issue, must be investigated in the upcoming days

This issue seems fixed in the last commit, the endpoint also seems working, except for the authentication issue linked in #19

Closing this issue here, any follow up to do in #19

@github-project-automation github-project-automation bot moved this from Urgent to Done in Solar Oct 24, 2023
@Nininanou16 Nininanou16 removed their assignment Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

1 participant