* Sets APP_PORT in env var of kubernetes yaml and sets default value in code if not set. Fixes#682
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* fixed whitespace in yaml for env var "value: "
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* fixing to point to public container image vs test image
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated hello-world/-kuberentes Node.js apps
1. Using async/await rather than promises for better readability
2. Make sure that methods do not return until state is committed
Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
* 💄
* moving devcontainer to root to enable
* Fix for tutorials to point to new dapr-cli-link
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed additional broken links found by MMD `make validate` in tutorials.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing readme to call python3 instead of python per best practices
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed zipkin api bug failing tests. Duration with current multiply code will never be slow enough to return traces results!
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Bump follow-redirects in /distributed-calculator/react-calculator/client (#530)
Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)
---
updated-dependencies:
- dependency-name: follow-redirects
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump follow-redirects in /pub-sub/react-form/client (#531)
Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)
---
updated-dependencies:
- dependency-name: follow-redirects
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
* Refresh pub-sub quickstart (tutorial) fixing routes and stale dependency (#539)
* Fixing error caused by route to * conflicing with /dapr/subscribe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrating from `requests` to `axios` to improve supportability
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changed from " to ' matching existing styles for Javascript
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing `axios` from import to require('axios'), matching require('express')
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Consistent use of Async enabled http code
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed react * route bug that conflicts with /dapr/subscribe route
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Reducing log levels introduced by migration. added debug flag.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating MM test to match improved precision of Express server.js with Axios.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* updating console.log from console.debug (minor) for consistency.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated vars to const, and then updated README to match.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Async posts needed await. Readme's tweaked to match updates.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Async should pair with await
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing comment
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Removing commented out pipe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in. Updated dependencies
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Paulyuk/fixpubsuberror (#547)
* Fixing error caused by route to * conflicing with /dapr/subscribe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrating from `requests` to `axios` to improve supportability
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changed from " to ' matching existing styles for Javascript
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing `axios` from import to require('axios'), matching require('express')
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Consistent use of Async enabled http code
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed react * route bug that conflicts with /dapr/subscribe route
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Reducing log levels introduced by migration. added debug flag.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating MM test to match improved precision of Express server.js with Axios.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* updating console.log from console.debug (minor) for consistency.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated vars to const, and then updated README to match.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Async posts needed await. Readme's tweaked to match updates.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Async should pair with await
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing comment
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Removing commented out pipe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in. Updated dependencies
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Hotfix to test, given precision sensitivity on test output number
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Improved MM test fix for quickstarts (#548)
* Fixing error caused by route to * conflicing with /dapr/subscribe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrating from `requests` to `axios` to improve supportability
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changed from " to ' matching existing styles for Javascript
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing `axios` from import to require('axios'), matching require('express')
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Consistent use of Async enabled http code
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed react * route bug that conflicts with /dapr/subscribe route
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Reducing log levels introduced by migration. added debug flag.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating MM test to match improved precision of Express server.js with Axios.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* updating console.log from console.debug (minor) for consistency.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated vars to const, and then updated README to match.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Async posts needed await. Readme's tweaked to match updates.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Async should pair with await
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing comment
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Removing commented out pipe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in. Updated dependencies
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Hotfix to test, given precision sensitivity on test output number
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing all remaining decimal precision issues with MM tests in readmes, including observability which now fails
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Fix curl sample to work on macOS (#543)
Signed-off-by: Marcin Jahn <marcinj10@gmail.com>
* Fix sample path in readme (#555)
Signed-off-by: Xin Shi <xshi@microsoft.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
* Remove PR check for docker login (#565)
* remove PR check for docker login
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
* remove PR trigger for the workflow to fix docker login issue
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
* Test validation cleanup (#569)
* Test validation cleanup
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
* add one response message in each of the validation steps
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
Co-authored-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Co-authored-by: Marcin Jahn <marcinj10@gmail.com>
Co-authored-by: Xin Shi <xshi@microsoft.com>
Co-authored-by: tanvigour <60332928+tanvigour@users.noreply.github.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
Francesco Belacca
Artur Souza
greenie-msft
Amulya Varote
Paul Yuknewicz
AkhilaC
amulyavarote
Alessandro (Ale) Segala
Doug Davis
Patrick Huber
Amulya Varote
Nick Greenfield