-
Notifications
You must be signed in to change notification settings - Fork 8
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
Port fixes for identity-apps #2482
Comments
Update [2024-02-02] - authenticationendpoint JSP file changesWorked on the analysis of modified jsp files in authenticationendpoint/includes. Please find the sheet containing the analysis :https://docs.google.com/spreadsheets/d/1TN_q5sTZwm-Wb1S_dhKppJpTPszqittAhRW-gVAEGPc/edit?usp=sharing |
Update [2024-02-06, 07] - authenticationendpoint JSP file changesCompleted the analysis of IS related changes (from IS patches) for 1.6.x branch. Concluded that there are no identity fixes to be added to the 1.6.x branch. Further identified several jsp files which are identical is IS and APIM, within authenticationendpoint/includes directory. Built the APIM pack, excluding those fiels (this included building the carbon apimgt, identity oauth saml2, carbon kernel and product apim with some orbit jars which are not yet relleased.) could build a product pack successfully, but the following error appears at the startup and the server stops. :
Triggered another build upgrading the h2 orbit version in carbon apimgt and carbon kernel. |
Update [2024-02-08 ] - authenticationendpoint JSP file changesStarted working on merging the APIM customizations with updated jsp files in identity apps into product apim Considered the diff which had been maintained in APIM 4.2.0 GA pack and the corresponding identity apps branch which was used there (1.5.0 tag). With that, merged the changes to product APIM, comparing it with updated identity apps branch (1.6.x) completed the analysis for half of the files EoD |
Update [2024-02-09 ] - authenticationendpoint JSP file changesIdentified another set of files other than the authenticationendpoint/includes, which needed to be synced with identity apps changes. Worked on and Completed the process of merging identity apps changes aiming the alpha release. With that, jsp changes for alpha release is done. PR : wso2/identity-apps#5461 Blocked in some build failures in product apim while trying to build a product pack with the changes (due to some other OSGI issues) In the progress of removing backup-code usages from identity apps repo. The process seems not straightforward because there are errors from transitively affected files. |
Update [2024-02-10 ] - authenticationendpoint JSP file changesRemoved backup code usage with PR : wso2/identity-apps#5492
When debugging the code, it was identified that, when it invokes the endpoint authAPIURL = https://localhost:9443/api/identity/auth/v1.1/data/AuthRequestKey/710dfce6-6460-449a-b2cd-5650ac0b1187, it returns the html page of the carbon console as contextProperties instead of the expected json response, at AuthParameterFilter.getServletRequestWithParams() in carbon-identity-framework. |
Update [2024-02-13, 14 ] - authenticationendpoint JSP file changesWhen checking the identity framework tag in APIM 4.2.0, AuthParameterFilter.java class is not there. It has been introduced with a new IS feature which causes to add an additional war file to APIM product. Considering the said issue it was decided to remove the corresponding fix from 1.6.x branch. Created the PR : wso2/carbon-identity-framework#5509 Considering the impact of the said fix and after having another discussion with the IS team, it was decided to keep the fix in identity apps branch and merge it into product apim as well. For M2 release it was targeted to keep the rest of the jsp changes + backup code removal. Tried figuring out the build issue occured while building the product. Built all relevant repos in a clean repo and could resolve the build issue. With that, built the APIM pack with jsp changes (not contain the security fix that caused the above issues), there was a problem in the loaded UI for login pages. fixed it by solving the name mismatch of a themes min css file. When click on the forget password option it returned a 400 status code. When checked the same flow in M1 pack, the issue exists there as well. Created the issue #2477 to track it. With that, created the following PRs completing the targetted progress for alpha release : |
Update [2024-02-15, 16, 19 ]Ported the features and tested the startup and initial login flows . PRs : changes are being added to PRs 2, 3, 4, 5 because they are not merged (added after m2 release) Completed the analysis of patches which has changes across multiple identity components with respect to the following jsp files :
Applied the corresponding patches and tested the basic flows in the product pack. |
Update [2024-02-20]Had to re-evaluated the issue wso2/product-is#14878 for applying the changes into APIM. Discussed to remove this feature only from product apim, while maintaining the relevant changes in identity apps and identity framework. It was identified that the corresponding war (api#identity#auth#v1.1 Evaluated the possible applications of IdentifierFirstLogin, including account creation process from devPortal. couldn't find a usage. Hence it was discussed to remove corresponding if clause considering the usage,wso2/product-apim@5b47361 Removed the corresponding code block from our code and tested the startup and basic login flows. |
Update [2024-02-21, 22]Analysed the security fixes for following files one by one and checked the changes made each commit. (fixes were applicable mainly in identity apps + carbon identity framework, identity inbound localaouth and kernel)
Applied the fixes which were missing in the corresponding branch. |
Update [2024-02-25 Sunday]Analysed the security fixes for following files one by one and checked the changes made each commit. (fixes were applicable mainly in identity apps + carbon identity framework, identity inbound localaouth and kernel)
With that, all the files in extensions directory in product APIM (customized files) and all the files in authenticationendpoint/includes are covered by the effort up to now. *** Please note that modifications from the patch updates which has been released after 22nd February 2024 are not included. Started checking on the updates to the set of jsp files of authenticationendpoint which are directly coming from identity apps (no customizations from APIM side and not in the includes/ directory) |
Update [2024-02-26]Completed applying identity apps related patches on all relevant IS branches. There's one update (U2 6779) which was discussed to reproduce and check the applicability before applying, because it requires to pack an additional war app inside the product. Tested flow :
But when we edit it and add country instead of organization, and press enter, it renders that corresponding preview altering the initial UI as follows : Since the issue is reproducible in APIM, decided to apply the fix. |
Added config and tested the pack without war file and component jars, to check the usage of the config added by update 6779:
|
Update [2024-02-27]Added the webapp and the 2 jars manually inside the product pack and tested the fix. Tested the fix and verified that the initially reproduced security vulnerability is addressed by the fix. |
Update [2024-02-29]It was decided to create a new feature module in Product APIM side, considering the abilitiy to create a new branch at identity-local-auth-api repo. commit : wso2/product-apim@0000e98 |
Closing as the task is completed. |
Initially this issue was tracked by #2428
The text was updated successfully, but these errors were encountered: