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

Fix CosmosDB state store handling of nulls #1974

Merged
merged 2 commits into from Aug 18, 2022

Conversation

berndverst
Copy link
Member

Addresses some issue related to the handling of nulls / nil values on Cosmos DB State store. #1970

  • No longer treats empty / nil values as binary data on Set operations.
  • Allows returning of nil values in Get operation.
  • Falls back to returning raw string if a byte string cannot be decoded (in Get operation).

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

Checklist

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

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

… data

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

/ok-to-test

@codecov
Copy link

codecov bot commented Aug 17, 2022

Codecov Report

Merging #1974 (5b1eb80) into master (4bc800f) will decrease coverage by 0.00%.
The diff coverage is 21.42%.

@@            Coverage Diff             @@
##           master    #1974      +/-   ##
==========================================
- Coverage   37.64%   37.64%   -0.01%     
==========================================
  Files         192      192              
  Lines       23951    23961      +10     
==========================================
+ Hits         9017     9019       +2     
- Misses      14169    14177       +8     
  Partials      765      765              
Impacted Files Coverage Δ
state/azure/cosmosdb/cosmosdb.go 10.43% <21.42%> (+0.20%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

ItalyPaleAle
ItalyPaleAle previously approved these changes Aug 17, 2022
Copy link
Contributor

@ItalyPaleAle ItalyPaleAle left a comment

Choose a reason for hiding this comment

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

Approved but with one comment

Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
@berndverst berndverst merged commit 1edd408 into dapr:master Aug 18, 2022
berndverst added a commit to berndverst/components-contrib that referenced this pull request Aug 18, 2022
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
mindovermiles262 pushed a commit to mindovermiles262/components-contrib that referenced this pull request Aug 18, 2022
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

2 participants