Skip to content

MODUSERBL-175: Remove legacy endpoint#212

Open
steveellis wants to merge 2 commits intomasterfrom
MODUSERBL-175-remove-legacy-tokens
Open

MODUSERBL-175: Remove legacy endpoint#212
steveellis wants to merge 2 commits intomasterfrom
MODUSERBL-175-remove-legacy-tokens

Conversation

@steveellis
Copy link
Contributor

https://folio-org.atlassian.net/browse/MODUSERBL-175

Summary of changes:

  • Remove authn/login from RAML
  • Return PostBlUsersLoginWithExpiryResponse instead of PostBlUsersLoginResponse from routes.
  • Update all legacy tests to use expiry endpoint
  • Revise module descriptor, bumping up major version and mod-login version

- RAML
- Remove usage of PostBlUsersLoginResponse
- Revise all tests to use expiry endpoin

TODO
Module desc
@steveellis steveellis changed the title MODUSERBL-175: Remove legacy tokens MODUSERBL-175: Remove legacy endpoint Feb 21, 2025
@steveellis
Copy link
Contributor Author

@gurleenkaurbp I added do not merge tag since we need to wait for mod-login and mod-authtoken to be merged, and for other modules to upgrade to mod-login's new interface so as to not break snapshot.

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
33.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

1 similar comment
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
33.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants