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

chore: [PAGOPA-2626] Decoupler: logic for FdR SOAP requests #2766

Merged
merged 26 commits into from
Feb 13, 2025

Conversation

aomegax
Copy link
Contributor

@aomegax aomegax commented Feb 4, 2025

  • Deploy FDR decoupler
  • Add Cache to nodoChiediFlussoRendicontazione

List of changes

Motivation and context

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Does this introduce a change to production resources with possible user impact?

  • Yes, users may be impacted applying this change
  • No

Does this introduce an unwanted change on infrastructure? Check terraform plan execution result

  • Yes
  • No

Other information


If PR is partially applied, why? (reserved to mantainers)

@aomegax aomegax requested review from a team as code owners February 4, 2025 21:42
@aomegax aomegax added env: dev The status refers to DEV environment fdr The changes refer to FdR domain labels Feb 4, 2025
@aomegax aomegax self-assigned this Feb 4, 2025
src/domains/fdr-app/04_apim_aux.tf Outdated Show resolved Hide resolved
src/domains/fdr-app/04_apim_aux.tf Outdated Show resolved Hide resolved
src/domains/fdr-app/04_apim_aux.tf Show resolved Hide resolved
src/domains/fdr-app/04_apim_aux.tf Show resolved Hide resolved
src/domains/fdr-app/99_variables.tf Show resolved Hide resolved
src/domains/fdr-app/99_variables.tf Outdated Show resolved Hide resolved
@pasqualespica
Copy link
Contributor

@aomegax @andrea-deri after this PR #2775 the apim policy on FDR API, where will they be? On pagopa-infra ( here ) or on pagopa-fdr pagopa/pagopa-fdr#82 ?

# Conflicts:
#	src/domains/fdr-app/04_apim_fdr_fase3.tf
#	src/domains/fdr-app/README.md
@aomegax
Copy link
Contributor Author

aomegax commented Feb 11, 2025

@aomegax @andrea-deri after this PR #2775 the apim policy on FDR API, where will they be? On pagopa-infra ( here ) or on pagopa-fdr pagopa/pagopa-fdr#82 ?

Yes, it regards FdR3 not FdR1 policies. It is right.

andrea-deri and others added 2 commits February 12, 2025 09:43
…ussiRendicontazione` primitive (#2781)

* [PAGOPA-2647] feat: defining response caching for FdR-Fase1 'flow list' API

* [PAGOPA-2647] fix: resolving bugs and refactoring code

* [PAGOPA-2647] fix: excluding caching process if response returns a fault code

* [PAGOPA-2647] fix: updating tracing text

* [PAGOPA-2647] fix: resolving bugs and adding correct handling of BLOB headers

* [PAGOPA-2647] fix: adding permission for read/write operations by APIM

* [PAGOPA-2647] chore: commenting trace tag, avoiding unnecessary log
@pasqualespica pasqualespica removed env: dev The status refers to DEV environment env: uat The status refers to UAT environment labels Feb 13, 2025
<!-- Execute a GET call on blob-storage using the UUID as "search filter" -->
<authentication-managed-identity resource="https://storage.azure.com/" output-token-variable-name="msi-access-token" ignore-error="false" />
<send-request mode="new" response-variable-name="cached_response_content" timeout="300" ignore-error="true">
<set-url>@{ return "https://{{fdr_cachedresponse_saname}}.blob.core.windows.net/" + "{{fdr_cachedresponse_containername}}" + "/" + ((string) context.Variables["fdr_cached_response_uuid"]) + ".txt"; }</set-url>
Copy link
Contributor

Choose a reason for hiding this comment

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

remove .txt with empty or .xml ext

@pasqualespica pasqualespica merged commit 152048f into main Feb 13, 2025
5 checks passed
@pasqualespica pasqualespica deleted the PAGOPA-2626-decoupler branch February 13, 2025 21:32
Copy link

🎉 This PR is included in version 1.386.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fdr The changes refer to FdR domain released status: applied ✋
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants