Skip to content

Fix FLOWISE-319 Sanitize User#6269

Open
yau-wd wants to merge 2 commits intomainfrom
fix/flowise-319-sanitize-user
Open

Fix FLOWISE-319 Sanitize User#6269
yau-wd wants to merge 2 commits intomainfrom
fix/flowise-319-sanitize-user

Conversation

@yau-wd
Copy link
Copy Markdown
Contributor

@yau-wd yau-wd commented Apr 22, 2026

FLOWISE-319

@yau-wd yau-wd self-assigned this Apr 22, 2026
@yau-wd yau-wd added the bug Something isn't working label Apr 22, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the dedicated login endpoint from the account controller and routes, refactors the AccountService to utilize a centralized sanitizeUser utility for cleaning user data, and updates the passport middleware to improve user name and email handling. A review comment suggests ensuring consistency in the AccountService by explicitly assigning the result of the sanitizeUser function back to the user object, matching the pattern used elsewhere in the file.

Comment thread packages/server/src/enterprise/services/account.service.ts Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
accountController.invite
)

router.post('/login', accountController.login)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are we not using this endpoint? how can user login if we remove this

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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants