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

[PAGOPA-1109] Authorizer - Adding metadata on authorization strings #13

Conversation

andrea-deri
Copy link
Contributor

@andrea-deri andrea-deri commented Aug 30, 2023

This PR contains all the update made in order to include the metadata information in the request to be passed to the Authorizer Policy-based's API. The inclusion of the metadata is required in order to save these information in the cached authorization string.
Other update made in this PR are related to a better optimization of the whole system (for example, replacing the comma with the hash character in order to remove a intermediate step of replacing in the Authorizer Policy-based's API).

List of Changes

  • Added metadata information on the request for Authorizer Policy-based's API.
  • Changed data passed as input for the requests on Authorizer Policy-based's API.

Motivation and Context

These changes are required in order to permit the caching of the metadata in the authorization string.

How Has This Been Tested?

  • Tested in DEV environment

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@andrea-deri andrea-deri added the enhancement New feature or request label Aug 30, 2023
@andrea-deri andrea-deri requested a review from a team August 30, 2023 14:14
@andrea-deri andrea-deri self-assigned this Aug 30, 2023
@sonarcloud
Copy link

sonarcloud bot commented Aug 31, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@pasqualespica pasqualespica merged commit d7fe401 into main Aug 31, 2023
4 of 5 checks passed
@andrea-deri andrea-deri deleted the PAGOPA-1109-sviluppo-autorizzatore-gestione-enti-pluri-intermediati branch September 15, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants