Skip to content

Conversation

@MaryanneNjeri
Copy link
Contributor

Context

This PR, remove any trailing forward slashes from store endpoint to make the tasks more resilient.


Task Name

  • AzureAppConfigurationImportV10
  • AzureAppConfigurationExportV10
  • AzureAppConfigurationSnapshotV1

Description

Remove any trailing forward slashes from store endpoint, to make the tasks more resilient. The tasks should work with or without a trailing slash. When a trailing slash is part of the endpoint the tasks fail with a 401.


Risk Assessment (Low / Medium / High)

Low


Change Behind Feature Flag (Yes / No)

No


Tech Design / Approach

  • Design has been written and reviewed.
  • Any architectural decisions, trade-offs, and alternatives are captured.

Documentation Changes Required (Yes/No)

No


Unit Tests Added or Updated (Yes / No)

Yes


Additional Testing Performed

Manual testing was performed


Logging Added/Updated (Yes/No)

  • Appropriate log statements are added with meaningful messages.
  • Logging does not expose sensitive data.
  • Log levels are used correctly (e.g., info, warn, error).

Telemetry Added/Updated (Yes/No)

No


Rollback Scenario and Process (Yes/No)

  • Rollback plan is documented.

Dependency Impact Assessed and Regression Tested (Yes/No)

  • All impacted internal modules, APIs, services, and third-party libraries are analyzed.
  • Results are reviewed and confirmed to not break existing functionality.

Checklist

  • Related issue linked (if applicable)
  • Task version was bumped — see versioning guide
  • Verified the task behaves as expected

@MaryanneNjeri MaryanneNjeri requested a review from a team as a code owner November 21, 2025 12:14
@MaryanneNjeri MaryanneNjeri force-pushed the user/maryanne/trim_slashes_store_endpoint branch from d93e1d7 to e63dc77 Compare November 21, 2025 12:33
@MaryanneNjeri
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@MaryanneNjeri MaryanneNjeri merged commit d867805 into master Nov 25, 2025
14 checks passed
@MaryanneNjeri MaryanneNjeri deleted the user/maryanne/trim_slashes_store_endpoint branch November 25, 2025 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants