Commit Graph

516 Commits

Author SHA1 Message Date
Ahson Khan
ad87699a2e
Disable unit tests for staticLibCurl since they are failing on Mac (#3068) 2021-11-05 14:18:10 -07:00
Azure SDK Bot
7926ec8c3c
Increment package version after release of azure-core (#3058) 2021-11-05 14:08:33 -07:00
Ahson Khan
e498d6a165
Update cspell.json entry for contributor names within azure core changelog. (#3049)
* Update cspell.json entry for contributor names within azure core changelog.

* Remove in-line exception
2021-11-04 15:31:04 -07:00
Victor Vazquez
4614b80d22
cl (#3047) 2021-11-04 18:52:50 +00:00
Victor Vazquez
50b89caaa5
hide Request general ctor (#3033) 2021-11-03 16:09:07 -07:00
Victor Vazquez
fb03127d05
Use ms for connection timeout (#3001)
* move the static transport adapter to private impl

* use MS for connection timeout

* update

* fix the value
2021-11-03 23:08:35 +00:00
Daniel Jurek
413b4b7665
Add coverage enforcement (#2992)
* Add coverage enforcement

* Add spelling exception for mspremier

* Use previous build coverage data, enforce 3% variance, use Skip.CoverageEnforcement feature flag

* Use version 8 of quality checks

* Remove forceCoverageImprovement, it doesn't allow 0% improvements (e.g. doc changes, etc.)

* branch -> branches

* Configure coverage thresholds per-service

* Use fixed coverage

* Review feedback

* Per-service coverage plumbing

* Flip branch and line coverage requirements for KV to accurately reflect current baseline
2021-11-03 10:56:32 -07:00
Victor Vazquez
31398df179
move the static transport adapter to private impl (#2993) 2021-10-28 11:49:37 -07:00
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