Skip to content

[Test Infra] adding operation-location to headers that are scrubbed #18514

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

Merged
merged 1 commit into from
May 5, 2021

Conversation

seankane-msft
Copy link
Member

Found when working with @iscai-msft that some services provide the next URL in the operation-location header, which can leak accounts and specific job details. Adding this to the headers that are scrubbed the same way location is.

@seankane-msft seankane-msft self-assigned this May 5, 2021
Copy link
Contributor

@iscai-msft iscai-msft left a comment

Choose a reason for hiding this comment

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

thanks @seankane-msft !

@seankane-msft
Copy link
Member Author

/azp run python - apconfiguration - ci

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@seankane-msft
Copy link
Member Author

/azp run python - apconfiguration

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@seankane-msft
Copy link
Member Author

/check-enforcer evaluate

@seankane-msft seankane-msft merged commit 0a58f4a into Azure:master May 5, 2021
@seankane-msft seankane-msft deleted the overload-location branch May 5, 2021 17:57
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request May 5, 2021
…into azure_purview_catalog

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  ignore analysis for swagger readmes (Azure#18520)
  [purview] add azure-purview-nspkg (Azure#18518)
  [AppConfiguration] Appconfig consistency (Azure#18493)
  [Container Registry] Improved samples (Azure#18263)
  [Container Registry] renamings (Azure#18492)
  [ServiceBus] internal code rename and sample readme update (Azure#18516)
  [EventHub] update link in sample readme (Azure#18517)
  Post Process Event Names Script (Azure#18419)
  [Tables] use etag from entity if match condition is given (Azure#18271)
  adding operation-location to headers that are scrubbed (Azure#18514)
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request May 6, 2021
…into azure_purview_scanning

* 'master' of https://github.com/Azure/azure-sdk-for-python: (550 commits)
  del useless files (Azure#18528)
  delete existing useless files for trafficmanager (Azure#18525)
  Define new replacer to replace keys in recording (Azure#18294)
  [purview] add purview nspkg to ci (Azure#18523)
  ignore analysis for swagger readmes (Azure#18520)
  [purview] add azure-purview-nspkg (Azure#18518)
  [AppConfiguration] Appconfig consistency (Azure#18493)
  [Container Registry] Improved samples (Azure#18263)
  [Container Registry] renamings (Azure#18492)
  [ServiceBus] internal code rename and sample readme update (Azure#18516)
  [EventHub] update link in sample readme (Azure#18517)
  Post Process Event Names Script (Azure#18419)
  [Tables] use etag from entity if match condition is given (Azure#18271)
  adding operation-location to headers that are scrubbed (Azure#18514)
  [Tables] Adds support for AzureNamedKeyCredential (Azure#18456)
  [Tables] delete_entity takes an entity instead of row and partition key (Azure#18269)
  [Tables] Removed TableEntity attribute wrapper (Azure#18489)
  [EventHub&ServiceBus] Bump uAMQP dependency (Azure#17942)
  [ServiceBus] add keyword override support to update_ methods in mgmt module (Azure#18210)
  Add compatibility switch to disable CAE (Azure#18148)
  ...
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Apr 12, 2022
[Azure Data Factory] Support rest-based connectors in ADF (Azure#18514)

* Updating Dataworld in custom words file

* Updating Linkedservice details with Dataworld information

* Update LinkedService.json

* Asana Swagger changes

* swagger changes for appfigures

* Update LinkedService.json

* Twilio linked service changes for datafactory

* Synapse linked service changes for Twilio

* Twilio custom-words.txt change

* Resolving conflicts

* removed extra comma

Co-authored-by: Aravind Reddy <[email protected]>
Co-authored-by: Priyadarshi Ghosh <[email protected]>
Co-authored-by: mshrivastavamsft <[email protected]>
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.

2 participants