Implementation of Form Authentication #155
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* management-api * Upgrades jolokia.js to same 2.0 version * Updates to reflect jolokia API
* First implementation assumed only Openshift. Adds form-based as well * Fronts with new OAuthService which reads the config and determines which delegate / protocol service should be initialised; either 'form' or 'openshift' * Maps out basics of FormService though not yet implemented but more importantly ensures that the existing OSOAuthService still correctly logs in and out.
* Extends webpack config to handle * env vars for either form or oauth authentication * routing via either external locations or react internal routes * Fixes hawtconfig logo url * form-service.ts * Fully implemented service that redirects to the login page if no token * .../login/*.tsx * Provides customised login component page that only has input for token so re-models the LoginForm to only have a token input textbox * index.ts * Adds Hawtio Plugin that acts as a login provider where the given component provide the login form * utils/fetch.ts * Extends fetchPath to add optional headers * tsup.config.ts / types.d.ts * Enables compilation and inclusion of svg and other media files * Adds jest testing scaffolding and JWT Decode tests
* Provides template functionality for hawtio-online app * Clones non-exported components from hawtio-next to ensure that form based login is working correctly
…wtio#151) * kubernetes-api * Small changes reflecting changes in handling of metadata values * kubernetes-api-app * Clones the changes made to the oauth-api-app to support login page from form authentication
…wtio#151) * management-api * Adds in error handling if kubernetes-api is in error state. Do not continue trying to get pods but return intialised. * This allows the /login page to properly be displayed with no uncaught thrown errors. Once login takes place then the redirect re-initialises eveything * management-api-app * Clones the changes made to the oauth-api-app to support login page from form authentication * Adds in colour styling for the pod management json - red if there is an error; blue otherwise * Adds possible explanations for possible failures to remind in the future possible avenues why things might be going wrong
* Adds scaffolding previously implemented in test-apps to enable form authentication
This was referenced Oct 12, 2023
Merged
|
Note: Tests will remain in a failed state until the release of a new version of hawtio-next that supports required changes in API. |
|
hawtio/hawtio-next#607 now merged. Needs to be updated with new release version of |
|
@phantomjinx hawtio/react v0.6.0 is already released so you can use it. However, I found an issue that it restarts the jolokia instance everytime the jolokia service getJolokia() is invoked. We need to release v0.6.1 soon that fixes it. |
Did you mean Updating hawtio-online presently. Thx! |
|
@phantomjinx Already fixed it with v0.6.1. |
phantomjinx
added a commit
to phantomjinx/hawtio-online
that referenced
this pull request
Oct 18, 2023
phantomjinx
added a commit
to phantomjinx/hawtio-online
that referenced
this pull request
Oct 18, 2023
phantomjinx
force-pushed
the
oauth-form
branch
from
01b2d2a
to
202b911
Compare
phantomjinx
added a commit
to phantomjinx/hawtio-online
that referenced
this pull request
Oct 18, 2023
phantomjinx
force-pushed
the
oauth-form
branch
from
202b911
to
f43ba40
Compare
phantomjinx
force-pushed
the
oauth-form
branch
from
f43ba40
to
2329c1f
Compare
|
Now upgraded dependency to @hawtio/react 0.6.1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merging of this is dependent on a version of hawtio-next that includes the PR #607 being released.