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

THREESCALE-8735: Client secret in clear text in the API Manager UI (RH SSO Integration) Part 2 #3856

Merged
merged 9 commits into from
Sep 2, 2024

Conversation

lvillen
Copy link
Contributor

@lvillen lvillen commented Jul 30, 2024

What this PR does / why we need it:
Currently, our admin SSO Integrations display the client secret when checking an integration as long as when editing or creating a new one.

Which issue(s) this PR fixes
THREESCALE-8735. This second part applies Patternfly and masks the client secret at the Admin SSO Integration new and edit forms.

Before
New
Captura de pantalla 2024-07-30 a las 17 31 17

Edit
Captura de pantalla 2024-07-30 a las 17 31 46

After
New
Captura de pantalla 2024-07-30 a las 17 29 48

Edit
Captura de pantalla 2024-07-30 a las 17 32 54

More details and snapshots to come related to developer portal changes

@lvillen lvillen self-assigned this Jul 30, 2024
@lvillen lvillen force-pushed the THREESCALE-8735_SSO-Integration-admin-side branch from 2ca0f61 to bf3ec00 Compare August 1, 2024 11:17
Copy link
Contributor

@jlledom jlledom left a comment

Choose a reason for hiding this comment

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

I tested it locally and works fine.

features/support/paths.rb Outdated Show resolved Hide resolved
config/locales/en.yml Show resolved Hide resolved
@lvillen lvillen force-pushed the THREESCALE-8735_SSO-Integration-admin-side branch 2 times, most recently from cdeed21 to 9aef015 Compare August 7, 2024 12:57
@lvillen lvillen marked this pull request as ready for review August 7, 2024 12:58
app/models/authentication_provider/keycloak.rb Outdated Show resolved Hide resolved
features/support/paths.rb Outdated Show resolved Hide resolved
features/support/paths.rb Outdated Show resolved Hide resolved
@mayorova mayorova force-pushed the THREESCALE-8735_SSO-Integration-admin-side branch from 9aef015 to 09fed84 Compare August 8, 2024 11:20
@lvillen lvillen force-pushed the THREESCALE-8735_SSO-Integration-admin-side branch from e187526 to ca36747 Compare August 30, 2024 10:10
jlledom
jlledom previously approved these changes Aug 30, 2024
@lvillen lvillen force-pushed the THREESCALE-8735_SSO-Integration-admin-side branch from d80e10e to eeaf89d Compare August 30, 2024 10:54
@lvillen lvillen requested a review from jlledom August 30, 2024 10:54
@lvillen lvillen merged commit c444155 into master Sep 2, 2024
17 of 21 checks passed
@lvillen lvillen deleted the THREESCALE-8735_SSO-Integration-admin-side branch September 2, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants