forked from mosip/mimoto
-
Notifications
You must be signed in to change notification settings - Fork 1
Develop #36
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
Open
kneckinator
wants to merge
726
commits into
idpass:develop
Choose a base branch
from
mosip:develop
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Develop #36
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
uocnb
approved these changes
Jul 21, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
Injiweb 1349 Update readme with uris
…pose (#550) Signed-off-by: Abhishek Paul <[email protected]>
Signed-off-by: Kamlesh Singh Bisht <[email protected]>
Signed-off-by: Kamlesh Singh Bisht <[email protected]>
…-token/{issuer} endpoints Signed-off-by: Gurpreet41082 <[email protected]>
Signed-off-by: Gurpreet41082 <[email protected]>
Injiweb 1359
Signed-off-by: Nandhukumar <[email protected]>
Signed-off-by: Gurpreet41082 <[email protected]>
Update develop from release
[INJIWEB-1167]: Multi-lingual support for credential download
Signed-off-by: Chandra Keshav Mishra <[email protected]>
MOSIP-38489 - Generated single report with 2 sections
Signed-off-by: NitinHegde <[email protected]>
…560) Signed-off-by: Praful Rakhade <[email protected]>
* [INJIMOB-1349] add data-share and minio service as part of docker compose Signed-off-by: Abhishek Paul <[email protected]> * [MOSIP-38411] rename main to master (#553) Signed-off-by: Chandra Keshav Mishra <[email protected]> * [INJIWEB-1349] removed endpoint /issuers/{issuer-id}/credentialTypes Signed-off-by: Gurpreet41082 <[email protected]> * [INJIWEB-1349] updated mimoto image in docker file Signed-off-by: Gurpreet41082 <[email protected]> * [DSD-7079] Updated action/upload artifact version push-trigger.yml (#559) * [DSD-] Updated action/upload artifact version push-trigger.yml Signed-off-by: Praful Rakhade <[email protected]> * Updated push-trigger.yml Signed-off-by: Praful Rakhade <[email protected]> --------- Signed-off-by: Praful Rakhade <[email protected]> --------- Signed-off-by: Abhishek Paul <[email protected]> Signed-off-by: Chandra Keshav Mishra <[email protected]> Signed-off-by: Gurpreet41082 <[email protected]> Signed-off-by: Praful Rakhade <[email protected]> Co-authored-by: Abhishek Paul <[email protected]> Co-authored-by: Chandra Keshav Mishra <[email protected]> Co-authored-by: Praful Rakhade <[email protected]>
Signed-off-by: techno-467 <[email protected]>
Signed-off-by: techno-467 <[email protected]>
Signed-off-by: Praful Rakhade <[email protected]>
[MOSIP-39276] separated helm and deployment script
Signed-off-by: techno-467 <[email protected]>
[MOSIP-39276] updated namespace from mimoto to injiweb
…it as part of the issuer configuration endpoint (#563) * [INJIWEB-1203] add an endpoint to fetch both issuers and authorization server wellknown and return to the caller Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] use caffiene cache provider to store the auth server wellknown response into cache and set the expiry time of cache entry in application properties file Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] add caffiene cache config in mimoto default properties file to use it when running the app using docker compose Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] deprecate the old endpoint of the issuers which is used to get the wellknown and add annotations with detailed info to include it in docs Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] use new exception message for auth server invalid wellknown scenario Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] in cache along with authorization endpoint and grant types store token endpoint from the auth server wellknown response Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] use token endpoint from authorization server wellknown as authorization audience Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] remove code related to fetching authorization_audience & proxy_token_endpoint from issuers config as we will be fetching them directly from auth server wellknown Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update issuers test case to use new endpoint configuration and see if it is returning the response as expected Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] use custom errors to throw exceptions for authorization server wellknown fetching related scenarios Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] write different test cases for authorization server service class Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] write test case to check if the getIssuerConfiguration method of issuersServiceImpl is giving the response as expected or not Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] add test case to check if validator library is throwing the error or not when authorization_servers in issuer wellknown is null Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] use getTokenResponse method of credentialService class in idpController to avoid redundant code Signed-off-by: PuBHARGAVI <[email protected]> * [INJWEB-1203] add new test cases in issuerServiceImpl class and udpate AuthorizarionServerService class tests Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update the authorization_servers in getIssuersConfig file to fix the failing tests Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update the issuers endpoints to fetch authorization_audience & proxy_token_endpoint from issuer wellknown and construct issuer wellknown based on credential_issuer_host field Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update issuers configuration endpoint to return the response in a response wrapper and update its test to check if the response is returned properly or not Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] add a new test case to check the error scenario of issuers configuration endpoint Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] change auth server impl file info log to debug log and add info about new endpoint in openapi json file Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] throw entire stack trace in auth server impl file exceptions and udpate the test cases Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] remove issuance part when constructing issuer wellknown and add jakarta dependency required for validations Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] remove authorization_audience, proxy_token_endpoint & wellknown_endpoint from mimoto issuer config file Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update the issuersDTO with constructed wellknown endpoint in issuers service impl file Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] add issuer_id & credential_issuer_host in docker compose issuer config file and replace resource issuers and default properties file with docker compose file contents Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] in tests store issuer wellknown and config response in json files and update the tests to fetch response from those files Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] add enable configuration annotation in cache config instead of main file Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] revert the changes of application-local properties file Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] remove unnecessary null checks & refactor the method names, add cache support for issuer wellknown endpoint and update the tests as per the PR comments Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] exclude authorization_audience in issuers endpoint response and expose & validate qr_code_type of issuer Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] extract getIssuerWellknown into separate class because spring doesn't provide cache support when cacheable method is called within the same class Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update issuer controller and service to remove redundant lines of code and methods and update the test cases accordingly Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] use object mapper of jackson library to deserialize issuer class and remove annotations related to gson & remove unused QrCOdeTypeDeserializer and update test cases Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] change authorization_audience annotation from url to not blank Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] updated open api documentation for issuers endpoints Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] remove the logic of remvoing authorization_audience in getIssuers method of issuer service as sending the new field in the response is not an issue and remove unused dependencies Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] update getIssuerConfigDTO method of test utilities as we are returning authorization_audience in all endpoints Signed-off-by: PuBHARGAVI <[email protected]> --------- Signed-off-by: PuBHARGAVI <[email protected]>
…s list if there is any issue while fetching the verifiers list (#569) Signed-off-by: PuBHARGAVI <[email protected]>
…ipOtp issuer (#570) * [INJIWEB-1203] ignore fetching wellknown for mosipOtp issuer Signed-off-by: PuBHARGAVI <[email protected]> * [INJIWEB-1203] set protocol value for issuers in test utilites Signed-off-by: PuBHARGAVI <[email protected]> --------- Signed-off-by: PuBHARGAVI <[email protected]>
#571) * [INJIWEB-1203] Updating handling of wellknown APis failure in issuers endpoint Signed-off-by: Gurpreet41082 <[email protected]> * [INJIWEB-1203] Fixed test case Signed-off-by: Gurpreet41082 <[email protected]> --------- Signed-off-by: Gurpreet41082 <[email protected]>
* MOSIP-36455 - Merge release 0.14.x branch with develop Signed-off-by: Mohanachandran S <[email protected]> * MOSIP-36455 - Merge release 0.14.x branch with develop Signed-off-by: Mohanachandran S <[email protected]> * INJIWEB-967 - Updated Automation Tests for Sunbird and eSignet Issues Signed-off-by: Mohanachandran S <[email protected]> * MOSIP-38392 - updated the apitest commons version Signed-off-by: Mohanachandran S <[email protected]> * MOSIP-38392 - Updated the issuer name for sunbird Signed-off-by: Mohanachandran S <[email protected]> * pulled latest changes Signed-off-by: Mohanachandran S <[email protected]> * Pulled latest changes Signed-off-by: Mohanachandran S <[email protected]> * Pulled latest changes Signed-off-by: Mohanachandran S <[email protected]> * updated the sunbird baseurl property Signed-off-by: Mohanachandran S <[email protected]> * updated the sunbird baseurl property Signed-off-by: Mohanachandran S <[email protected]> * updated the sunbird baseurl property Signed-off-by: Mohanachandran S <[email protected]> * updated the sunbird baseurl property Signed-off-by: Mohanachandran S <[email protected]> --------- Signed-off-by: Mohanachandran S <[email protected]>
Signed-off-by: Gurpreet41082 <[email protected]>
[injiweb-1510] added support to generate signing keys for multiple algorithms and store them in database during the wallet creation
…update the tests accordingly Signed-off-by: PuBHARGAVI <[email protected]>
…ntials Signed-off-by: PuBHARGAVI <[email protected]>
…ponse dto into separate factory class Signed-off-by: PuBHARGAVI <[email protected]>
…f any of the encrypted fields are updated Signed-off-by: PuBHARGAVI <[email protected]>
…oad or fetching credentials then throw the error Signed-off-by: PuBHARGAVI <[email protected]>
…ys from the DB to sign the JWT in credential request except for rsa Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
…algorithm of a issuer directly in credentialUtilService file Signed-off-by: PuBHARGAVI <[email protected]>
…rithm to generate key pair for signing and verifying the jwt of credential request in without login flow and for login flow use the keys from db for all algorithms Signed-off-by: PuBHARGAVI <[email protected]>
Signed-off-by: PuBHARGAVI <[email protected]>
…ims in jwt of credential request to 5 hours and write tests for jwtGeneratorUtil file Signed-off-by: PuBHARGAVI <[email protected]>
[Injiweb 1418] add APIs to download and store the credential into the database and to fetch the credentials of a specific wallet from the database in login flow
…ocumentation file and also update the incorrect platform error codes for some scenarios and udpate the test cases for the same Signed-off-by: PuBHARGAVI <[email protected]>
…api documentation file Signed-off-by: PuBHARGAVI <[email protected]>
…tation file and update some descriptions Signed-off-by: PuBHARGAVI <[email protected]>
[Injiweb 1418] add info about login endpoints in openapi documentation json file
Signed-off-by: Gurpreet41082 <[email protected]>
Signed-off-by: Gurpreet41082 <[email protected]>
… deploy properties file and remove an invalid comment in verifiable credentials table Signed-off-by: PuBHARGAVI <[email protected]>
…cript [INJIWEB-1418] fix the issue of DML_FLAG property not being read from deploy properties file and remove an invalid comment in verifiable credentials table
[Injiweb 1461] add support to run the login flow using docker setup
…eady downloaded or not for particular issuers to check it based on walletId, issuer and credentialType Signed-off-by: PuBHARGAVI <[email protected]>
* Adding release 17 documentation (#670) * Adding release 17 documentation Signed-off-by: Gurpreet41082 <[email protected]> * corrected the file in which cache properties are present Signed-off-by: Gurpreet41082 <[email protected]> --------- Signed-off-by: Gurpreet41082 <[email protected]> * [INJIMOB-3140] update logic in issuersValidConfig file to stop the validation as soon as an error is found for any issuer (#647) * [INJIMOB-3140] in issuersValiationConfig file throw the exception immediately if any validations fail for an issuer rather than proceeding further to check for other issuers Signed-off-by: PuBHARGAVI <[email protected]> * [INJIMOB-3140] update the validation logic in issuersValidationConfig file to accumulate all the issuers errors and then throw the error in the end Signed-off-by: PuBHARGAVI <[email protected]> * [INJIMOB-3140] add missing annotations in the issuerDTO file and write test cases for issuerValidationConfig file Signed-off-by: PuBHARGAVI <[email protected]> --------- Signed-off-by: PuBHARGAVI <[email protected]> * updated readme from release branch Signed-off-by: Gurpreet41082 <[email protected]> --------- Signed-off-by: Gurpreet41082 <[email protected]> Signed-off-by: PuBHARGAVI <[email protected]> Co-authored-by: PuBHARGAVI <[email protected]>
[Injiweb 1418] fix download issue observed for certain issuer
Signed-off-by: Kamlesh <[email protected]>
…ger.yml Signed-off-by: Praful Rakhade <[email protected]>
[DSD-7833] removed mimoto-apitest publish-to-nexus job from push-trig…
[INJIWEB-1505]: Added jti field
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.
No description provided.