* Policies do not overwrite HTTP headers if they are already set
* Use static constant UserAgent
---------
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* add documentation for cadl sync and generate
* move the file to doc/common and link to it from each script
* update links after file rename
---------
Co-authored-by: m-nash <prognash@gmail.com>
* Dummy commit to get the current CI error
* Touch file in core to trigger the CI
* Dummy commit to get the current CI error
* Touch file in core to trigger the CI
* Wonderful, and a broken link now too
* Undo any yml changes (Fixed externally?)
* Undo all intentionally unnecessary changes
---------
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* move scripts to common
* add reference to common script
* address feedback
* couple tweaks to relative path updates
* update copy paste issue with emitter path
* updates to make scripts more generic
---------
Co-authored-by: m-nash <prognash@gmail.com>
* Consolidate naming logic and generate short hash names for local use
* Shorten long lines
* Handle issues with EnvironmentVariable parameter ref being updated
* Warn on env variable overwrite. Base name generation off resource group
* Use SHA256 algorithm for short name hash
---------
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* feature/upload-proxy-artifact
* update template to ensure that having the file in use on windows doesn't break the build
* incoming PR fixes where test-proxy stores its log. as as result, we also need to update the location of where we read out the test-proxy log
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Ignore warning C4100 on Windows for unused parameter in OpenTelemetry
headers.
* Disable other warnings like 4244 and 6323 as well.
* Clang format fix.
* Add CI pipeline step and script that validates all files in the repo contain ASCII-only bytes and no UTF-8 BOM
* Update platform-matrix.json to turn on CHECK_ASCII_CHARACTERS
* Fix missing comma in json.
* Update indentation in the ci.test.yml for the bash script
* Exclude the .github folder and test recording json files.
* Remove "recordings" exclusion since it doesn't exist anymore.
* Fixed UTF-8 with BOM and other non-ASCII character issues within the
files in the repo.
* Exclude nlohmann json tests from ASCII validation check.
* Log an intro to the CI console for the validation step.
* Remove codecoverage.cmake exclusion.
* Remove exclusion for docs and tools directories.
- Made Context immutable (including marking the static ApplicationContext as const
- Put back TelemetryPolicy in all pipeline cases and removed User-Agent generation from RequestActivityPolicy (note: This part of the change may end up being reverted). Also updated parameters for TelemetryPolicy to make it clearer that the parameter is a package name, not a service name.
- Changed Az.Namespace value from being the package name to being the service name.
- Change test SpanExporter to fully capture exported values rather than keeping references to the values.
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
* Experiment with CI and MultipleCrlOperations test
* Switch to an older, less failure-proof version of the test from yesterday
* Switch Ubuntu18 to Azure Pipelines as well
* Undo CI changes just to make sure it fails
* Apply CI machine pool changes
* Move Ubuntu20 to general pool
* Move Ubuntu20 to general pool
* Move Ubuntu20 to perf pool
* Revert all changes in platform-matrix.json
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
* skeletor
* client
* rework
* all works plus test
* sample , self pr
* tests
* clang
* Mind your language .. Sir ...
* comments
* removed preprocessor directives
* removed template class
* dll clang
* Rick PR
* Create ApiViewSettings for most existing packages
* Corrected name for blobs storage review name
* Added cspell settings for DCURL
* Create ApiViewSettings for most existing packages
* Corrected name for blobs storage review name
* Added cspell settings for DCURL
* ApiView settings changes for DataLake
* Renamed includeInternal to allowInternal; Added storage common; Fixed package names
* Fixed spelling error
* If there's an error accessing the site with no CRL checks, don't try it any more
* Stop checking example.com because it doesn't work
* Update sdk/attestation/azure-security-attestation/inc/ApiViewSettings.json
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
* Update sdk/core/azure-core/inc/ApiViewSettings.json
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
* Update sdk/keyvault/azure-security-keyvault-secrets/inc/ApiViewSettings.json
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
