Move OAuth2 endpoints to OAuth2 controller
refactor: Move OAuth2 endpoints to OAuth2 controller (#3450)
Move OAuth2 endpoints to OAuth2 controllerrefactormove oauth2 endpoints controller
Committed by ahsanv
commit message
refactor: Move OAuth2 endpoints to OAuth2 controller (#3450)
* Move oauth2 endpoints to oauth2 controller
* Remove old oauth2-credential auth endpoint from server.ts
* Move OAuth2 callback endpoint to controller
* Fix tests and eslint issues
* Fix typo
* fix lint issues
* update package-lock
* Import lodash methods individually
* Minimise lint rule disables
* Cleanup
* rebase
* CR
* npm package: Remove lodash, use lodash.intersect
* fixups
* rebaseEditor assessment
The header is acceptable — refactor without scope, a capitalized 'Move' where lowercase imperative 'move' is preferred, and a (#3450) reference — but the body betrays its squashed origin. It is a raw transcript of fifteen PR commits, including trivia like 'rebase,' 'CR,' 'Cleanup,' and 'fixups,' with CRLF endings; that is commit archaeology, not a coherent description of the refactor. Notably, none of it explains what moving OAuth2 endpoints to a controller achieves or what might break. A scope such as (core) or (oauth2) would help, and a few summarizing bullets would transform this message.