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

[AutoPR sql/resource-manager/v2014_04_01] Add TDEBYOK api swagger files for serveroperations and MIoperations #777

Conversation

openapi-sdkautomation[bot]
Copy link

vhvb1989 and others added 30 commits September 23, 2019 10:58
* Changed default retry timeout and extended test jitter bounds to fix transient tests

* Extending retry bounds more
* cosmos current version update

* remove smoke-tests change from PR

* use the project.version to set the properties version
* Upgrade to latest version of reactor core

* Update reactor-core version in azure-core

* Upgrade reactor-core to 3.2.12

* Add automatic module name to spotbugs pom

* Add automatic module names to eng modules
* Use version based profiles to compile groovy code in encryption clients

* Removed configuration argLine values

* Fix retry delay calculation
…izer. Add EventHubClient.getEventHubName (#5474)

* Add MessageSerializer interface.

* Update documentation in EventData.

* Rename EventDataUtil -> EventHubMessageSerializer.

* Exposing EventHubClient.getEventHubName() as public.

* Remove ManagementResponseMapper and consolidate into MessageSerializer.

* Removing static public method from EventHubMessageSerializer.

* Fix build breaks when passing MessageSerializer to EventHubAsyncProducer.

* Adding tests for EventHubMessageSerializer. Fixing NullPointerException.

* Move MessageSerializer into azure-core-amqp.
* Add module-info to appconfig

* Add license header

* Add license header

* Add exports to app config
update SB SDK to mention Management client functionality
* Move Reactor handlers into azure-core-amqp.

* Move ReactorDispatcher, AmqpErrorCode to azure-core-amqp.

* Add correct requires in module-info.java.

* Moving implementation specific classes to azure-core-amqp.

* Move tests into azure-core-amqp.

* Use Mockito instead of concrete class.

* Rename host to hostname.

* Fix usages of FQDN.
* Fixed the setter and getter method

* Remvoe setter
* Fixed the setter and getter method
* Convert credential scopes param to TokenRequest

* Fix build errors in event hubs

* Change .block() to StepVerifier

* Fix tests

* checkstyle .* imports
* remove java code isImple check but move to suppression and add only check for public class for external Dependency check

* fix issue 5113
* Rename from values to getValues()
* Peering: generate 2019 08 01 preview

* fix: compile error
conniey and others added 20 commits October 21, 2019 21:54
* Fix broken copy tests.

* Swap delaySubscription with share(), to fix activationOperation being invoked twice.

* Fix copy tests to use asynchronous copy.

* Fix Abort copy lease.

* Fixing copy tests with new test records..

* Fix container tests.
* Added test for empty parallel upload

* Handle empty buffers

* Use hasReamining instead of greater than 0
* Removed linting suppressions, added documentation, and annotated fluent and immutable classes

* Fix typo and few naming casings
* Change from contentMD5 -> contentMd5.

* Change from toURL -> toUrl. Change from checked exception to unchecked.

* Change from getKeySHA256 -> getKeySha256.

* Fix casing for URL.

* Fix parameter names.

* change from MD5 to Md5.

* Ignoring broken tests and adding a maximum duration.
* Preview 4 change log

* Add README and samples for user auth

* Add 1.0.0 change log

* Change date & Add known issues
* Enable exponential backoff for retry policy in http pipeline

* Fix checkstyle

* minor refactoring

* Update testcases

* Update retry policy based on PR feedback

* Fix javadoc
* Update documentation in exceptions.

* Update documentation in HTTP classes.

* Update documentation in policies.

* Adding headers to PagedFlux

* Update documentation and snippets in PagedFluxBase.
* Fix documentation

* fix package name
#5943)

* Added UTF-8 encoding for blob names in methods that are used to build blob clients or manipulate blobs. Also added documentation where blob names are part of the flow so that the user knows what is happening in case they are thinking of passing an already encoded name.

* Added missing comments about encryption to the Javadoc of some classes. Removed the use of the Utility.urlEncoder() in a couple methods where it was not necessary and would actually cause double encoding.  Added one use of said method to BlobAsyncClient.

* Changed a few couple Groovy tests that were failing because the expected session records did not account for the newly added encoding

* Removed a call to Utility.urlEncode() in BLobContainerAsyncClient that would make blob URLs to be encoded twice.

* Broke a couple lines in BlobBatch into two parts to comply with the code style guidelines.

* Added missing space to a line in BlobBatch. It was flagged by the CI style checker.

* Fixed an issue that made some test cases fail: blobName needs to be encoded when putting together the blobUrl that is passed as a a constructor argument for specialized blob clients. Added blobName decoding before encoding it so that if users pass encoded names we can properly deal with them; little to no overhead was added by this.

* Made sure that blob names are decoded from UTF-8 before being returned by getters and modified certain builders to account for this. Added encoding for setting the blobName in BlobServiceSasSignatureValues. Modified and added tests to account for encoding and added the generated session records as well. Corrected a test that did not compile on BlockBlobAPITest.

* Because we are doing encoding blob names implicitly, there is no point in telling the user we are doing this, so I removed parts of the Javadoc that referenced the fact. For the sake of clarity, I added the the Javadoc in blob name getters so that users know the returned blob name will always be decoded (it's possible they set an encoded name and expect to see the same value from the get method). Also added a clarification that states blob names must be encoded to UTF-8 to the Javadoc of methods that take blob URLs as an argument.

* Made sure that the blobName used to generate a SAS signature is decoded. According to Rick Ley from the Storage team: "storage account name and the resource name must be URL-decoded".
@openapi-sdkautomation
Copy link
Author

Installation Instructions

You can install the package sql/resource-manager/v2014_04_01 of this PR by downloading the artifact jar files. Then ensure that the jar files are on your project's classpath.

Direct Download

The generated package artifacts can be directly downloaded from here:

Copy link

Hi @openapi-sdkautomation[bot]. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

2 similar comments
Copy link

Hi @openapi-sdkautomation[bot]. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

Copy link

Hi @openapi-sdkautomation[bot]. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

Copy link

Hi @openapi-sdkautomation[bot]. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing /reopen if you'd like to continue working on these changes. Please be sure to use the command to reopen or remove the no-recent-activity label; otherwise, this is likely to be closed again with the next cleanup pass.

@github-actions github-actions bot closed this Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.