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

azure appconfig configuration store #1874

Merged
merged 9 commits into from Jul 20, 2022
Merged

Conversation

cmendible
Copy link
Member

@cmendible cmendible commented Jul 11, 2022

Signed-off-by: cmendible cmendible@gmail.com

Description

Implementing azure appconfig configuration store.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

This PR refers to #1541 but does not close it as it doesn't fully implement all API surfaces

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@cmendible cmendible requested review from a team as code owners July 11, 2022 21:01
@cmendible cmendible force-pushed the issue-1541 branch 4 times, most recently from 955e1e4 to f6c1c65 Compare July 11, 2022 21:44
@cmendible cmendible force-pushed the issue-1541 branch 3 times, most recently from 4f23a2e to f0bff5c Compare July 12, 2022 07:56
Copy link
Member Author

@cmendible cmendible left a comment

Choose a reason for hiding this comment

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

Done!

configuration/azure/appconfig/appconfig.go Outdated Show resolved Hide resolved
internal/authentication/azure/auth.go Outdated Show resolved Hide resolved
internal/authentication/azure/auth.go Outdated Show resolved Hide resolved
@cmendible cmendible force-pushed the issue-1541 branch 5 times, most recently from 40af21a to e6e7239 Compare July 16, 2022 08:36
Signed-off-by: cmendible <cmendible@gmail.com>
@berndverst
Copy link
Member

Is this still work in progress? LGTM.

@cmendible cmendible changed the title [WIP] azure appconfig configuration store azure appconfig configuration store Jul 19, 2022
Signed-off-by: cmendible <cmendible@gmail.com>
Signed-off-by: cmendible <cmendible@gmail.com>
Copy link
Contributor

@mukundansundar mukundansundar left a comment

Choose a reason for hiding this comment

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

please add unit tests for max retry settings parse errors.

configuration/azure/appconfig/appconfig.go Outdated Show resolved Hide resolved
configuration/azure/appconfig/appconfig.go Outdated Show resolved Hide resolved
configuration/azure/appconfig/appconfig.go Show resolved Hide resolved
configuration/azure/appconfig/appconfig_test.go Outdated Show resolved Hide resolved
configuration/azure/appconfig/appconfig_test.go Outdated Show resolved Hide resolved
configuration/azure/appconfig/appconfig_test.go Outdated Show resolved Hide resolved
configuration/azure/appconfig/appconfig_test.go Outdated Show resolved Hide resolved
internal/authentication/azure/auth.go Show resolved Hide resolved
Signed-off-by: cmendible <cmendible@gmail.com>
Signed-off-by: cmendible <cmendible@gmail.com>
Signed-off-by: cmendible <cmendible@gmail.com>
@codecov
Copy link

codecov bot commented Jul 20, 2022

Codecov Report

Merging #1874 (c40eca7) into master (5743229) will increase coverage by 0.03%.
The diff coverage is 27.69%.

@@            Coverage Diff             @@
##           master    #1874      +/-   ##
==========================================
+ Coverage   36.81%   36.85%   +0.03%     
==========================================
  Files         181      182       +1     
  Lines       16337    16449     +112     
==========================================
+ Hits         6015     6062      +47     
- Misses       9624     9693      +69     
+ Partials      698      694       -4     
Impacted Files Coverage Δ
bindings/alicloud/dingtalk/webhook/webhook.go 53.84% <0.00%> (ø)
bindings/alicloud/rocketmq/rocketmq.go 0.00% <0.00%> (ø)
bindings/alicloud/tablestore/tablestore.go 2.54% <0.00%> (ø)
bindings/aws/sqs/sqs.go 6.34% <0.00%> (-0.21%) ⬇️
bindings/azure/blobstorage/blobstorage.go 17.64% <ø> (ø)
bindings/azure/cosmosdb/cosmosdb.go 18.00% <ø> (ø)
...ngs/azure/cosmosdbgremlinapi/cosmosdbgremlinapi.go 8.88% <0.00%> (ø)
bindings/azure/eventgrid/eventgrid.go 3.62% <0.00%> (-0.23%) ⬇️
bindings/azure/eventhubs/eventhubs.go 14.53% <0.00%> (-0.26%) ⬇️
...indings/azure/servicebusqueues/servicebusqueues.go 2.02% <0.00%> (ø)
... and 33 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91ce3d3...c40eca7. Read the comment docs.

Copy link
Contributor

@mukundansundar mukundansundar left a comment

Choose a reason for hiding this comment

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

lgtm for the initial Get implementation.

Copy link
Contributor

@mukundansundar mukundansundar left a comment

Choose a reason for hiding this comment

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

@berndverst can we not close #1541 till the Subscribe and Unsubscribe methods are implemented?

@ItalyPaleAle
Copy link
Contributor

@mukundansundar I changed the first post so it won't close the issue automatically.

@berndverst berndverst merged commit 20e987b into dapr:master Jul 20, 2022
@berndverst berndverst added this to the v1.9 milestone Jul 20, 2022
mindovermiles262 pushed a commit to mindovermiles262/components-contrib that referenced this pull request Aug 18, 2022
* azure appconfig configuration store

Signed-off-by: cmendible <cmendible@gmail.com>

* added retry options

Signed-off-by: cmendible <cmendible@gmail.com>

* removed unused property

Signed-off-by: cmendible <cmendible@gmail.com>

* fixed error msgs, typos and test names

Signed-off-by: cmendible <cmendible@gmail.com>

* resource documenation

Signed-off-by: cmendible <cmendible@gmail.com>

* return err from not implemented functions

Signed-off-by: cmendible <cmendible@gmail.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Loong Dai <long.dai@intel.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>
berndverst added a commit that referenced this pull request Aug 19, 2022
* Azure AD support in SignalR (#1852)

* WIP: Azure AD support in SignalR

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* Correct SignalR AAD details

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* Misc fixes

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* azauth package name

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* use docker for linting for github action compatability

Signed-off-by: Andrew <mindovermiles262@gmail.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* update makefile to check version rather than using docker

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* try windows again

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Pin certification tests to v1.8.0-rc.4

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Added conformance test for Azure Blob Storage state store

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* state.azure.blobstorage now passes conformance tests

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Added more logging to state conformance tests

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Reporting ETag failures correctly in delete method

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Fixing #1845

Signed-off-by: cmendible <cmendible@gmail.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Pin certification tests to dapr v1.8.0

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Ft/sasl scram sha 256 512 (#1856)

* working with hardcoded SHA-512

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* cleanup code

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Do not hardcode specific testTopicName

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Ensure context propagation in MySQL binding (#1829)

Spin-off from PR adding contexts to input bindings

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Add support for AAD auth in Azure Storage Queues binding (#1842)

* Add support for AAD auth in Azure Storage Queues binding

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* 🧹

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Moved authentication to be an internal pkg (#1855)

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Azure AD support in SignalR (#1852)

* WIP: Azure AD support in SignalR

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* Correct SignalR AAD details

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* Misc fixes

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* azauth package name

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* rename SCRAM properly as SASL

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* update gomod/sum

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* gofmt

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* mod tidy

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* goval

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* use xdg-go instead of depreicated xdg library

Signed-off-by: Andrew Duss <andy.duss@storable.com>

Co-authored-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Updated devcontainer for 1.8

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Fixes 400 error empty content from sendgrid (#1867)

Signed-off-by: Ben Coleman <benc.uk@gmail.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* optimize: add context to huawei/obs (#1820)

* optimize: add context to huawei/obs

Signed-off-by: 1046102779 <seachen@tencent.com>

* optimize: add context to huawei/obs

Signed-off-by: 1046102779 <seachen@tencent.com>

* optimize: add context to huawei/obs

Signed-off-by: 1046102779 <seachen@tencent.com>

Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Add contexts to input bindings (#1831)

This is the last part of the "shutdown sequence fix": it allows shutting down input bindings before output ones.

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Updated pinned version of dapr/dapr in cert tests (#1877)

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* pubsub.redis: better ack for messages (#1886)

Also do not log errors for "context canceled" during shutdown

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Add user agent to AWS requests (#1890)

* Add user agent to AWS requests

Signed-off-by: Clare Liguori <liguori@amazon.com>

* Fix import spacing

Signed-off-by: Clare Liguori <liguori@amazon.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* pubsub.mqtt: support wildcard and shared subscriptions (#1882)

Fixes #1881

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Added pubsub.FeatureSubscribeWildcards capability (#1887)

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Calculate Cert test coverage and notify discord monitoring channel (#1857)

* Calulate Cert Test Coverage and Notify Discord Server

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

* Mentioning Monitoring Webhook Url

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

* Addressing comments

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

* Add warning comment on low coverage

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Job NOT to fail, even when Discord Norification fails

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

* Revert "Add warning comment on low coverage"

This reverts commit 35d4742.

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

* Fix Errors

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

* Not failing anywhere in Final Covergae Step

Signed-off-by: Deepanshu Agarwal <deepanshu.agarwal1984@gmail.com>

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* azure appconfig configuration store (#1874)

* azure appconfig configuration store

Signed-off-by: cmendible <cmendible@gmail.com>

* added retry options

Signed-off-by: cmendible <cmendible@gmail.com>

* removed unused property

Signed-off-by: cmendible <cmendible@gmail.com>

* fixed error msgs, typos and test names

Signed-off-by: cmendible <cmendible@gmail.com>

* resource documenation

Signed-off-by: cmendible <cmendible@gmail.com>

* return err from not implemented functions

Signed-off-by: cmendible <cmendible@gmail.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Loong Dai <long.dai@intel.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* modify error msg (#1897)

Signed-off-by: 1046102779 <seachen@tencent.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Use unique message key in Azure ServiceBus PubSub (#1898)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Do not skip duplicate actions for conf and cert tests

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Remove pre-condition from workflow (#1903)

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* pubsub.mqtt: support multiple wildcards in subscriptions (#1901)

Fixes #1881

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Certification test for Azure Blob storage state store component (#1896)

* Certification test for Azure Blob storage state store component

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* fix comments

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* fix comments

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* rebase and run mod tidy

Signed-off-by: tanvigour <tanvi.gour@gmail.com>

* Update Readme

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Added conformance tests for rabbitmq bindings (#1910)

* Added conformance tests for rabbitmq bindings

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>

* Fixing lint issues

Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* pubsub.snssqs: Support regions in other AWS partitions (#1925)

Fixes #1907

Signed-off-by: Clare Liguori <liguori@amazon.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Updated CI: (#1918)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* make linter timeout 15 min (#1931)

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* feat(pubsub/jetstream): allow tls client authentication (#1924)

Signed-off-by: NickLarsenNZ <nick@aptiv.co.nz>

Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Fix Apache Dubbo indirect dependency (#1934)

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Updated codecov Action to v3 and reduce email spam (#1936)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Ensure we're using the right version of cenkalti/backoff (#1904)

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Add auto-completion flag for Zeebe (#1908)

Signed-off-by: Christian Kaps <ck-github@mohiva.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* add ipfs binding (#1849)

* add web3.ipfs binding

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* Replaced go module as old versions brought in a dep with LGPL license

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* lint

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* changed cid -> path

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* add "ls" command

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* pin-ls returns cids

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* just "bindings.ipfs"

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* updated library to kubo

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* added tests

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

* linted

Signed-off-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>

Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Ensure rethinkdb complies with state.TransactionalStore (#1930)

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Implemented get all keys (#1920)

Signed-off-by: cmendible <cmendible@gmail.com>

Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Certification tests for env/file secrets stores #1761 (#1846)

* Certification tests for env/file secrets store #1761

Signed-off-by: cmendible <cmendible@gmail.com>

* using testing setEnv

Signed-off-by: cmendible <cmendible@gmail.com>

* removing nestedSeparator to test default behaviour

Signed-off-by: cmendible <cmendible@gmail.com>

* make modtidy-all

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Remove unnecessary replace in go.mod s

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* add a new binding component : alicloud sls log storage (#1873)

* new a bingding componet alicloud sls logstorage

Signed-off-by: RcXu <honeyxhl@163.com>
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* update sls binding component

Signed-off-by: RcXu <honeyxhl@163.com>
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* update the metadata decode method

Signed-off-by: RcXu <honeyxhl@163.com>
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* make modtidy-all

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* sort imports

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Co-authored-by: RcXu <rongcai.xu@ivydad.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* binding.eventhub: use same name schema for storage leases like on pubsub.eventhub (#1940)

* binding.eventhub: use same name schema for storage leases like on pubsub.eventhub

Signed-off-by: Christian Leinweber <christian.leinweber@maibornwolff.de>

* binding.eventhub: rename AzureEventshub local var

Signed-off-by: Christian Leinweber <christian.leinweber@maibornwolff.de>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* bindings.cron: ensure it can resume correctly after context cancellation (#1943)

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Adding in track2 SDK support for CosmosDB Bindings (#1876)

* Adding in track2 SDK support for CosmosDB Bindings

Signed-off-by: Ryan Lettieri <ryanLettieri@microsoft.com>

* Removing non needed ID check

Signed-off-by: Ryan Lettieri <ryanLettieri@microsoft.com>

* Addressing container read operation

Signed-off-by: Ryan Lettieri <67934986+RyanLettieri@users.noreply.github.com>

* Updating linting

Signed-off-by: Ryan Lettieri <67934986+RyanLettieri@users.noreply.github.com>

Co-authored-by: Ryan Lettieri <ryanLettieri@microsoft.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* updated local secret stores certification test dependencies (#1948)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Enable CosmosDB Binding conformance test (#1946)

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Allow more folks to run `/ok-to-test` (#1944)

Give access to `/ok-to-test` to several more people working on certification tests.

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Bump github.com/open-policy-agent/opa from 0.40.0 to 0.42.0 (#1945)

* Bump github.com/open-policy-agent/opa from 0.40.0 to 0.42.0

Bumps [github.com/open-policy-agent/opa](https://github.com/open-policy-agent/opa) from 0.40.0 to 0.42.0.
- [Release notes](https://github.com/open-policy-agent/opa/releases)
- [Changelog](https://github.com/open-policy-agent/opa/blob/main/CHANGELOG.md)
- [Commits](open-policy-agent/opa@v0.40.0...v0.42.0)

---
updated-dependencies:
- dependency-name: github.com/open-policy-agent/opa
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* make modtidy-all

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Enable conformance test for memcached state store (#1949)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Removing the use of the ioutil package And Fix CVE-2021-42576 (#1954)

* fix CVE-2021-42576

Signed-off-by: qiangmzsx <qiangmzsx@gmail.com>

* Removing the use of the ioutil package

Signed-off-by: qiangmzsx <qiangmzsx@gmail.com>

* Update middleware/http/nethttpadaptor/nethttpadaptor.go

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Signed-off-by: qiangmzsx <qiangmzsx@gmail.com>
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* fix daprbot (#1956)

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Ensure locks in channels are properly released (#1957)

Fixes #1865

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* fix: bindings.ipfs unit test (#1961)

Signed-off-by: 1046102779 <seachen@tencent.com>

Signed-off-by: 1046102779 <seachen@tencent.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* optimize: add e2etests buildtags for e2e (#1951)

* optimize: skip fatal when pubsub/jetstream server don't start in unit test

Signed-off-by: 1046102779 <seachen@tencent.com>

* optimize: skip fatal when pubsub/jetstream server don't start in unit test

Signed-off-by: 1046102779 <seachen@tencent.com>

* optimize: skip fatal when pubsub/jetstream server don't start in unit test

Signed-off-by: 1046102779 <seachen@tencent.com>

* optimize: add e2etests buildtags for e2e

Signed-off-by: 1046102779 <seachen@tencent.com>

Signed-off-by: 1046102779 <seachen@tencent.com>
Co-authored-by: Loong Dai <long.dai@intel.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Handle service bus exported errors (#1942)

* Add service bus exported error handling

Signed-off-by: Addison Juarez <adjuarez@microsoft.com>

* Add service bus exported error handling

Signed-off-by: Addison Juarez <adjuarez@microsoft.com>

Signed-off-by: Addison Juarez <adjuarez@microsoft.com>
Co-authored-by: Loong Dai <long.dai@intel.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* pubsub.in-memory: add support for wildcard topics (#1966)

* pubsub.in-memory: add support for wildcard topics

Fixes #1964

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>

* make modtidy-all

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Update code to reflect renaming on UnlockResponse.Status (#1958)

PR dapr/dapr#4989 (issue dapr/dapr#4988) updated the names of some
enums in `UnlockResponse.Status` ProtoBuff definition:

* `LOCK_UNEXIST` became `LOCK_DOES_NOT_EXIST `
* `LOCK_BELONG_TO_OTHERS`  became `LOCK_BELONGS_TO_OTHERS`

Code implementing Distributed Lock component in components-contrib
needs to be updated to account for this modification.

This PR accomplishes this by updating contants mapping back to the
two enums above to match their new names.

Fixes components-contrib#1950

Signed-off-by: Tiago Alves Macambira <tmacam@burocrata.org>

Signed-off-by: Tiago Alves Macambira <tmacam@burocrata.org>
Co-authored-by: Loong Dai <long.dai@intel.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* fix(kafka): ConsumeClaim should not return err when retry recovery failed (#1965)

* fix(kafka): ConsumeClaim should not return err when retry recovery failed

Signed-off-by: zcong1993 <zhangcong1992@gmail.com>

* chore: tweak error log message

Signed-off-by: zcong1993 <zhangcong1992@gmail.com>

Signed-off-by: zcong1993 <zhangcong1992@gmail.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Allow metadata to flow through service bus queue (#1911)

* Allow metadata to flow through service bus queue

This commit lets metadata bind to a message and be read when
the message is received. This allows for the distributed tracing
of events.

Signed-off-by: Hal Spang <halspang@microsoft.com>

* Remove saved fields from ApplicationProperties

Signed-off-by: Hal Spang <halspang@microsoft.com>

* Add a certification test for metadata

Signed-off-by: Hal Spang <halspang@microsoft.com>

Signed-off-by: Hal Spang <halspang@microsoft.com>
Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Fix CosmosDB state store handling of nulls  (#1974)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Go 1.19 support and linter fixes (#1975)

* Go 1.19 support and linter fixes

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Update workflows for Go1.19 and new linter version

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Remove unnecessary space in nolint directive

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* disable additional linters which aren't used because of Go generics

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* enable gosec linter again

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Update bindings/zeebe/command/publish_message_test.go

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Update bindings/output_binding.go

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Use prepared statement for mysql table creation

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Ping is not ping

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* c'mon linter

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* Fix MySQL gosec issue

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

* revert mysql to be fixed later

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>

* Fixes to MySQL state store (#1978)

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* instruct user to download if linter not installed

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* separated linter installed and version checking into targets

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* cleanup file

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* fixed bug where make would choke when linter not installed

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* fix typos

Signed-off-by: Andrew Duss <andy.duss@storable.com>

* add specific version of linter to install

Signed-off-by: Andrew Duss <andy.duss@storable.com>

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Signed-off-by: Andrew Duss <andy.duss@storable.com>
Signed-off-by: Andrew <mindovermiles262@gmail.com>
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: cmendible <cmendible@gmail.com>
Signed-off-by: Ben Coleman <benc.uk@gmail.com>
Signed-off-by: Clare Liguori <liguori@amazon.com>
Signed-off-by: Amulya Varote <amulyavarote@microsoft.com>
Signed-off-by: Christian Leinweber <christian.leinweber@maibornwolff.de>
Signed-off-by: qiangmzsx <qiangmzsx@gmail.com>
Signed-off-by: 1046102779 <seachen@tencent.com>
Signed-off-by: Addison Juarez <adjuarez@microsoft.com>
Signed-off-by: Tiago Alves Macambira <tmacam@burocrata.org>
Signed-off-by: zcong1993 <zhangcong1992@gmail.com>
Signed-off-by: Hal Spang <halspang@microsoft.com>
Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Co-authored-by: cmendible <cmendible@gmail.com>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
Co-authored-by: Ben Coleman <benc.uk@gmail.com>
Co-authored-by: yellow chicks <seachen@tencent.com>
Co-authored-by: Clare Liguori <liguori@amazon.com>
Co-authored-by: deepanshuagarwal <deepanshu.agarwal1984@gmail.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Co-authored-by: Loong Dai <long.dai@intel.com>
Co-authored-by: Joni Collinge <jonathancollinge@live.com>
Co-authored-by: tanvigour <60332928+tanvigour@users.noreply.github.com>
Co-authored-by: amulyavarote <91495930+amulyavarote@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Christian Kaps <307006+akkie@users.noreply.github.com>
Co-authored-by: hypernova3 <108718961+hypernova3@users.noreply.github.com>
Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
Co-authored-by: Mr.Black <honeyxhl@163.com>
Co-authored-by: RcXu <rongcai.xu@ivydad.com>
Co-authored-by: Christian Leinweber <christian.leinweber@googlemail.com>
Co-authored-by: Ryan Lettieri <67934986+RyanLettieri@users.noreply.github.com>
Co-authored-by: Ryan Lettieri <ryanLettieri@microsoft.com>
Co-authored-by: Carlos Mendible <266546+cmendible@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: qiangmzsx <qiangmzsx@gmail.com>
Co-authored-by: addjuarez <addiajuarez@gmail.com>
Co-authored-by: Tiago Alves Macambira <macambira@gmail.com>
Co-authored-by: cong <zhangcong1992@gmail.com>
Co-authored-by: halspang <70976921+halspang@users.noreply.github.com>
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.

None yet

6 participants