Commit Graph

508 Commits

Author SHA1 Message Date
Victor Vazquez
871b9e8d6d
Expose http request base constructor (#3002)
* Expose http request base constructor

* cl
2021-10-28 10:02:01 -07:00
Victor Vazquez
3be793d558
[Core] - Support azure-core version without OpenSSL dependency (#2839)
* perf test uuid

* use win impl for uuid on linux

* use c99 base64 encode-decode
2021-10-27 23:20:34 +00:00
Victor Vazquez
a4f2e510e5
Adding timeout option (#2991) 2021-10-25 12:12:53 -07:00
Victor Vazquez
f90ab8a9b5
Add Storage Performance tests for DownloadTo API (#2987)
Adding 3 new tests for Storage Performance
The next tests are expected to find out the overhead between each layer of the Azure SDK client while performing a download operation. The current performance test is measuring the client + azure core pipeline + http transport.

The next tests removes authentication policy and then each layer up to the HTTP transport adapter directly.

DownloadBlobSas
Generates a SaS token to download a blob. This way the authentication policy "sharedKey" won't run in the pipeline.

DownloadBlobWithPipelineOnly
Use the SaS token as well but send the request without the BlobClient. Instead it uses the Azure Core Pipeline directly

DownloadBlobWithTransportOnly
Same as before, the SaS token is used to download the blob but consuming only the libcurl-transport adapter. This option skips the HttpPipeline. The only elements used from Azure Core are the HttpRequest and the Azure::Context

Note
porting tests from the .NET equivalent:

https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/storage/Azure.Storage.Blobs/perf/Azure.Storage.Blobs.Perf/Scenarios/DownloadSasUriBlobClient.cs

https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/storage/Azure.Storage.Blobs/perf/Azure.Storage.Blobs.Perf/Scenarios/DownloadSasUriHttpClient.cs

https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/storage/Azure.Storage.Blobs/perf/Azure.Storage.Blobs.Perf/Scenarios/DownloadSasUriHttpPipeline.cs
2021-10-25 11:48:55 -07:00
Victor Vazquez
ef72f3c8a1
expose Curl nosignal opt (#2985)
* add NoSignalOption

* cl

* tests
2021-10-21 17:25:02 -07:00
Victor Vazquez
dcc6f12c20
force http 1.1 on the libcurl transport adapter (#2978)
* force http 1.1

* update wording
2021-10-20 18:43:28 +00:00
Anton Kolesnyk
6c0901b2ff
Fix build warning in no-RTTI config (#2981)
* Fix build warning in no-RTTI config

* Fix build warning in no-RTTI config

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
2021-10-19 16:09:06 -07:00
Anton Kolesnyk
b477373a9c
Remove extra semicolons (#2979)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
2021-10-19 16:08:42 -07:00
Victor Vazquez
5e3cd1fb01
Adding a static-blocking libcurl transport adapter (#2948)
* static libcurl transport adapter

* ignore cspell

* updates to implementation
2021-10-18 15:01:00 -07:00
Victor Vazquez
8f3263a7ec
Missing fix for testing windows w/o RTTI (#2969)
* Enable no-rtti

* Apply suggestions from code review

Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>

* pass target

* rename

* rename

* fix for installed header

* update

* compile with tests

* add unit test for CI

* rename for each package

* wording updates

* renames

* rename

* update name

* fix for testing context pre-condition

Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
2021-10-18 15:00:47 -07:00
Victor Vazquez
f4e6390dff
Enable no-rtti (#2963)
* Enable no-rtti
2021-10-14 18:12:03 -07:00
Rick Winter
bb8b889cf5
Remove static_cast (#2815)
* Remove static_cast

* Add additional checks around our Logger::Level to ensure its lock_free
2021-09-16 11:54:18 -07:00
Azure SDK Bot
2b3f03909e
Increment package version after release of azure-core (#2826) 2021-09-02 20:56:31 +00:00
Victor Vazquez
14a4459ae0
core cl update (#2819) 2021-09-02 11:02:14 -07:00
Anton Kolesnyk
3aee1859f0
CSpell fixes for Core and Identity (#2816)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
2021-08-31 15:21:17 -07:00
Victor Vazquez
49636a2f93
ignore result of function to compile on G++ 5 (#2784)
* ignore result of function

* changelog

* Add g++5 to CI legs

* use inline config for g++ 5

* update flag

* set default values for variables

* fix format
2021-08-24 15:55:46 -07:00
Victor Vazquez
28d7a22fbd
support cmake version 3.12 for Azure core (#2787)
* support cmake version 3.12 for Azure core
2021-08-24 15:17:06 -07:00
Anton Kolesnyk
f1ae8a9cdf
Rename exceptions message parameter to whatArg (#2782)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
2021-08-23 18:41:16 -07:00
JinmingHu
a9507ae88d
Fix build error on some platforms (#2734)
* Move uuid impl to .cpp file

* CL
2021-08-13 07:50:25 +08:00
Victor Vazquez
b3cfe0148e
Playback test impl (#2737)
* checkpoint

* playback working

* formant
2021-08-12 10:58:21 -07:00
Victor Vazquez
b0cb94702c
supresss gtest warning from test pw (#2726) 2021-08-10 17:58:22 +00:00
Victor Vazquez
0c041c9bfa
Add core-test recording-playback (#2685)
Recording feature
2021-08-09 15:38:57 -07:00
Azure SDK Bot
4e1cdb985d
Increment package version after release of azure-core (#2715) 2021-08-09 13:20:01 -07:00
Anton Kolesnyk
6bf52bdbde
Update vcpkg version dependencies (#2718) 2021-08-09 13:15:27 -07:00
Victor Vazquez
2b63bc6536
changelog for 1.2.0 release (#2713) 2021-08-05 20:47:51 -07:00
Anton Kolesnyk
7f6930678c
Update changelog to list Clang 11 community contribution (#2707)
* Update changelog for Clang 11 comunity contribution

* Move to correct place, arrange in correct order

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
2021-08-04 10:33:29 -07:00
David Chisnall
eb135023bb
Fix implicit conversion that clang rejects. (#2704)
With clang 11, the build fails with:

```
azure-sdk-for-cpp/sdk/core/azure-core/inc/azure/core/datetime.hpp:285:14: error: no viable conversion from returned value of type 'time_point<Azure::_detail::Clock, typename common_type<duration<long, ratio<1, 10000000> >, duration<long long, ratio<1, 1> > >::type>' (aka 'time_point<Azure::_detail::Clock, duration<long long, ratio<__static_gcd<ratio<1, 10000000>::num, ratio<1, 1>::num>::value, __static_lcm<ratio<1, 10000000>::den, ratio<1, 1>::den>::value> > >') to function return type 'Azure::DateTime'
      return DateTime(1970) + std::chrono::seconds(posixTime);
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
2021-08-03 13:13:16 -07:00
Victor Vazquez
2feed76d86
clang doc fixes (#2697) 2021-08-02 11:55:08 -07:00
Victor Vazquez
554feb13b8
typo (#2700) 2021-08-02 11:54:56 -07:00
JinmingHu
b1c8ea92dc
Change the order of some built-in policies and client policies (#2670) 2021-07-29 10:07:09 +08:00
Anton Kolesnyk
326af88078
Make WinHTTP transport adapter to NOT use SSL/TLS for unsecured HTTP connections (#2678)
* Make WinHTTP transport adapter to NOT use SSL/TLS for unsecured HTTP connections

* Update changelog for identity as well

* LocaleInvariantCaseInsensitiveEqual + micro-optimization to avoid creating&allocating std::string each time

* Update sdk/identity/azure-identity/CHANGELOG.md

Co-authored-by: Rick Winter <rick.winter@microsoft.com>

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
2021-07-28 10:55:54 -07:00
Victor Vazquez
5c14975957
remove prefer ninja (#2675) 2021-07-27 19:26:11 -07:00
Victor Vazquez
a508789c36
Remove warning supress (#2677)
* remove warning suppress not required anymore

* json update

* update the curl header including

* changelog
2021-07-27 18:09:52 -07:00
Victor Vazquez
0567145212
support connection close header (#2672) 2021-07-27 10:03:56 -07:00
George Arama
0d4f138cb3
GetSecret Api implementation for Keyvault Secret (#2656)
* Progress stream reader

* format

* Update sdk/core/azure-core/src/io/body_stream.cpp

Co-authored-by: JinmingHu <jinmhu@microsoft.com>

* PR comments

* remove

* one more comment

* replaced if null with azure_assert

* moved from pointer to reference

* first pass

* src builds

* new line

* huzaaaaa

* readme

* strating point

* get progress, need to deserialize now

* serializer

* some more

* tests

* more tests

* some refactor

* start

* comment and formatting

* working again

* couple of updates

* clang

* remove pedantic ;

* include

* pr comments

* clang format

* trigger

* trigger 2

* PR comments

* name value swap

Co-authored-by: JinmingHu <jinmhu@microsoft.com>
2021-07-23 17:36:04 -07:00
George Arama
1f15e046e0
Progress stream reader (#2563)
* Progress stream reader

* format

* Update sdk/core/azure-core/src/io/body_stream.cpp

Co-authored-by: JinmingHu <jinmhu@microsoft.com>

* PR comments

* remove

* one more comment

* replaced if null with azure_assert

* moved from pointer to reference

* PR comments

* clang

Co-authored-by: JinmingHu <jinmhu@microsoft.com>
2021-07-20 13:28:48 -07:00
Anton Kolesnyk
89222f1950
Remove curl version from vcpkg manifest (#2572) 2021-07-10 01:44:01 -07:00
Azure SDK Bot
0a6a5080a9
Increment package version after release of azure-core (#2553) 2021-07-09 14:33:56 -07:00
George Arama
e8efcd21fd
2559 doc warnings (#2573)
* fix doc warnings

* format
2021-07-07 15:23:22 -07:00
JinmingHu
a8d7a48a07
Fix Bug: libcurl transport layer hangs on 304 response (#2558) 2021-07-06 18:28:32 +00:00
Victor Vazquez
f2413d11f5
update change log (#2551)
* update change log

* update change log
2021-07-02 11:03:11 -07:00
Anton Kolesnyk
6e4e9fd54b
Prepare Core and Identity releases (#2527)
* Prepare Core and Identity releases

* One more file

* Move to Other Changes

* Apply suggestions from code review

Co-authored-by: Ahson Khan <ahkha@microsoft.com>

* identity bugs fixed header

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
2021-07-01 16:06:36 -07:00
Ahson Khan
863e9871e5
Make sure RFE Message is only set to the response body payload. (#2539)
* Make sure RFE Message is only set to the response body payload.

* Add a note about the Message property.

* Jeff likes wordsmithing things for fun.
2021-07-01 15:56:20 -07:00
Ahson Khan
d0ca42643f
Move Sha256Hash and other Hash algorithm types from KeyVault to Azure::Core (#2526)
* Move the SHA256, 384, and 512 Hash implementations to be internal.

* Update changelog and add back missing file.

* Rename SHA256 and others to Sha256Hash and update header name.

* Fix up path in CMakeList by removing quotes.

* Rename the sha.cpp source file to match header.

* Move Sha256Hash and other Hash algorithm types from KeyVault to
Azure::Core.

* Reorder src file in cmakelist to be alpha order just to reset CI

* Revert "Reorder src file in cmakelist to be alpha order just to reset CI"

This reverts commit 6729cf311af76bb8388738cc519ea40092bc362c.
2021-07-01 15:42:36 -07:00
Ahson Khan
731d07c2cc
Fix up the implementation details of the RequestFailedException ctor. (#2536) 2021-07-01 14:08:05 -07:00
Ahson Khan
a2f027f0df
Disable C28020 warning in json.hpp as a stop gap on Windows. (#2535) 2021-07-01 14:04:16 -07:00
Ahson Khan
a6f3b530d5
Move Base64Url helper internal methods from KeyVault to Azure Core. (#2528)
* Move Base64Url helper internal methods from KeyVault to Azure Core.

* Run clang format.
2021-07-01 12:24:38 -07:00
Anton Kolesnyk
d4e347b4f1
Core Readme: reorder Key Concepts, remove Distributed Tracing (#2532)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
2021-07-01 11:00:19 -07:00
Anton Kolesnyk
3f5c5ae664
Managed Identity Credential (#2448) 2021-07-01 09:16:26 -07:00
George Arama
c449e3f9bc
2502 exceptions (#2529)
* first pass

* move exception creation from keyvault to core

* merge fix

* format

* fix includes location and containers

* remove default constructor

* to trigger build

* undo change

* uncomment

* Update sdk/core/azure-core/test/ut/exception_test.cpp

Co-authored-by: Rick Winter <rick.winter@microsoft.com>

Co-authored-by: Rick Winter <rick.winter@microsoft.com>
2021-07-01 04:17:15 +00:00