Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Basket.API project failed to use ByPassAuthMiddleware #1495

Closed
anjoy8 opened this issue Oct 20, 2020 · 13 comments · Fixed by newrelic-forks/eShopOnContainers#8
Closed

Basket.API project failed to use ByPassAuthMiddleware #1495

anjoy8 opened this issue Oct 20, 2020 · 13 comments · Fixed by newrelic-forks/eShopOnContainers#8
Labels

Comments

@anjoy8
Copy link
Contributor

anjoy8 commented Oct 20, 2020

If the authentication center identity.API is not enabled,
when using the manual default user test, the Claim is reported as an error.
图片

Because when we add a test user in the middleware(file : /src/Services/Basket/Basket.API/Infrastructure/Middlewares/ByPassAuthMiddleware.cs), the claim Type of the user Name is 'name':

new Claim("name", "Test user"),
new Claim("http://schemas.xmlsoap.org/ws/2005/05/identity/claims/surname","User"),

But in BasketController CheckoutAsync method, using ClaimType is (http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name),

var userName = this.HttpContext.User.FindFirst(x => x.Type == ClaimTypes.Name).Value;

the two do not match.

@anjoy8
Copy link
Contributor Author

anjoy8 commented Oct 20, 2020

On the other hand,
I think currentUserId should be assigned to a claims type named sub,
in which case, via /noauth?userid=xxx, which matches directly with the buyerId correspondence.

new Claim("sub", "1234"),
new Claim("sub", currentUserId),// <-- note this

@anjoy8
Copy link
Contributor Author

anjoy8 commented Oct 28, 2020

Could you please reply? Thank you. Do I need to submit a PR?

@sughosneo
Copy link
Contributor

Hi @anjoy8, thank you for reaching out.

Sorry couldn't get a chance to look into this. Will test it out and update you soon.

@sughosneo
Copy link
Contributor

Hi @anjoy8,

I have spent sometime to look into this issue and tried to reproduce it at my end but unfortunately to do so.
Could you please share more information on the details steps you have followed at your end on using ByPassAuthMiddleware ?

Please note, we have known issue on OrderingScenarios.Cancel_basket_and_check_order_status_cancelled where similar NullReferenceException exception can be seen due to missing claim type to the Invoke method of FunctionalTests.Middleware.AutoAuthorizeMiddleware. By "manual default user test" did you try to refer the similar things ?

Thank you

@anjoy8
Copy link
Contributor Author

anjoy8 commented Oct 30, 2020

The steps are as follows:

1. Download the source code for the Dev branch

图片

2. The basket.API project starts by default

图片

3. Configure EventBus and Redis ConnectionString and set UseLoadTest to true

{
  "Serilog": {
    "MinimumLevel": {
      "Default": "Debug",
      "Override": {
        "Microsoft": "Warning",
        "Microsoft.eShopOnContainers": "Debug",
        "System": "Warning"
      }
    }
  },
  "IdentityUrlExternal": "http://localhost:5105",
  "IdentityUrl": "http://localhost:5105",
  "ConnectionString": "xxx.25.251.xx,password=xxxx!",
  "AzureServiceBusEnabled": false,
  "EventBusConnection": "xxx.25.251.xx",
  "EventBusUserName": "xxx",
  "EventBusPassword": "xxxx!",
  "UseLoadTest": true

}

4. Start Kestrel mode

图片

5. Configure test user's userid = '123456798'

https://localhost:5103/noauth?userid=123456798
图片

6. Add a shopping cart

图片

7, Checkout user '123456798' shopping cart

{
  "city": "string",
  "street": "string",
  "state": "string",
  "country": "string",
  "zipCode": "string",
  "cardNumber": "string",
  "cardHolderName": "string",
  "cardExpiration": "2020-10-30T02:47:02.882Z",
  "cardSecurityNumber": "string",
  "cardTypeId": 1,
  "buyer": "123456798",
  "requestId": "3fa85f64-5717-4562-b3fc-2c963f66afa6"
}

8. An exception error occurred

图片

图片

@sughosneo
Copy link
Contributor

Hi @anjoy8 , thank you for providing this detailed information. Will test at my end and update you.

@sughosneo
Copy link
Contributor

Hi @anjoy8 , thank you for providing this detailed information. Will test at my end and update you.

Hi @anjoy8, thank you for providing the details. I have reviewed that code and details at my end.
Yes, you are right that's a bug. And that's because of the wrong value of the name and sub claim which subsequently gets set in line 63: and line 68: in ByPassAuthMiddleware. And because of that method Task<ActionResult> CheckoutAsync in BasketController class throws _identityService.GetUserIdentity() gives you back the wrong userId and Null refernce exceptions.

Would you like to submit a PR with the changes in ByPassAuthMiddleware ?

Thank you.

@anjoy8
Copy link
Contributor Author

anjoy8 commented Nov 4, 2020

Of course! I am willing to do it. I have submitted PR:#1508

In addition, I am now making some video about this architecture and promoting it in the community. I also really want to participate in the development and maintenance work. I don't know if it is ok.
(https://www.bilibili.com/video/BV1nT4y1L7vd)

@sughosneo
Copy link
Contributor

Of course! I am willing to do it. I have submitted PR:#1508

In addition, I am now making some video about this architecture and promoting it in the community. I also really want to participate in the development and maintenance work. I don't know if it is ok.
(https://www.bilibili.com/video/BV1nT4y1L7vd)

Hi @anjoy8, thank you for the PR.

Great to see your video and different contribution. 👍
Please note, your contribution is important and valuable to us and for the community.

You can look into the list of open issues and try to fix whichever is possible. Items marked as help wanted could be a good place to start. I will also try to ping you in different PRs for suggestions or validations.
Please feel free to share your thoughts, ideas and feedback.

Thanks again !

@anjoy8
Copy link
Contributor Author

anjoy8 commented Nov 4, 2020

Ok!
I love this architecture and will try to fix some open issues

alanwest added a commit to newrelic-forks/eShopOnContainers that referenced this issue Nov 13, 2020
* Fixed dotnet-architecture#1495 bug : UseLoadTest missing claimsType (dotnet-architecture#1508)

* Update ByPassAuthMiddleware.cs

* Update ByPassAuthMiddleware.cs

* remove WebMVC/ByPassAuthMiddleware duplicate cliam

* remove Ordering.API/ByPassAuthMid  duplicate cliam

* remove Location.API/ByPassAuthMid  duplicate cliam

* remove Market.API/ByPassAuthMid  duplicate cliam

* change webmvc bypassauth: claimType of sub

* change ordering bypassauth: claimType of sub

* change location bypassauth: claimType of sub

* change market bypassauth: claimType of sub

* Update solution items (dotnet-architecture#1515)

Co-authored-by: Manuel Cañete <[email protected]>

* Updated solution file. (dotnet-architecture#1514)

Co-authored-by: ansonzhang <[email protected]>
Co-authored-by: m-knet <[email protected]>
Co-authored-by: Manuel Cañete <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>
alanwest added a commit to newrelic-forks/eShopOnContainers that referenced this issue Nov 13, 2020
* Fixed dotnet-architecture#1495 bug : UseLoadTest missing claimsType (dotnet-architecture#1508)

* Update ByPassAuthMiddleware.cs

* Update ByPassAuthMiddleware.cs

* remove WebMVC/ByPassAuthMiddleware duplicate cliam

* remove Ordering.API/ByPassAuthMid  duplicate cliam

* remove Location.API/ByPassAuthMid  duplicate cliam

* remove Market.API/ByPassAuthMid  duplicate cliam

* change webmvc bypassauth: claimType of sub

* change ordering bypassauth: claimType of sub

* change location bypassauth: claimType of sub

* change market bypassauth: claimType of sub

* Update solution items (dotnet-architecture#1515)

Co-authored-by: Manuel Cañete <[email protected]>

* Updated solution file. (dotnet-architecture#1514)

Co-authored-by: ansonzhang <[email protected]>
Co-authored-by: m-knet <[email protected]>
Co-authored-by: Manuel Cañete <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>

Co-authored-by: ansonzhang <[email protected]>
Co-authored-by: m-knet <[email protected]>
Co-authored-by: Manuel Cañete <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>
alanwest added a commit to newrelic-forks/eShopOnContainers that referenced this issue Nov 13, 2020
* Fixed dotnet-architecture#1495 bug : UseLoadTest missing claimsType (dotnet-architecture#1508)

* Update ByPassAuthMiddleware.cs

* Update ByPassAuthMiddleware.cs

* remove WebMVC/ByPassAuthMiddleware duplicate cliam

* remove Ordering.API/ByPassAuthMid  duplicate cliam

* remove Location.API/ByPassAuthMid  duplicate cliam

* remove Market.API/ByPassAuthMid  duplicate cliam

* change webmvc bypassauth: claimType of sub

* change ordering bypassauth: claimType of sub

* change location bypassauth: claimType of sub

* change market bypassauth: claimType of sub

* Update solution items (dotnet-architecture#1515)

Co-authored-by: Manuel Cañete <[email protected]>

* Updated solution file. (dotnet-architecture#1514)

Co-authored-by: ansonzhang <[email protected]>
Co-authored-by: m-knet <[email protected]>
Co-authored-by: Manuel Cañete <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>
alanwest added a commit to newrelic-forks/eShopOnContainers that referenced this issue Nov 13, 2020
* Fixed dotnet-architecture#1495 bug : UseLoadTest missing claimsType (dotnet-architecture#1508)

* Update ByPassAuthMiddleware.cs

* Update ByPassAuthMiddleware.cs

* remove WebMVC/ByPassAuthMiddleware duplicate cliam

* remove Ordering.API/ByPassAuthMid  duplicate cliam

* remove Location.API/ByPassAuthMid  duplicate cliam

* remove Market.API/ByPassAuthMid  duplicate cliam

* change webmvc bypassauth: claimType of sub

* change ordering bypassauth: claimType of sub

* change location bypassauth: claimType of sub

* change market bypassauth: claimType of sub

* Update solution items (dotnet-architecture#1515)

Co-authored-by: Manuel Cañete <[email protected]>

* Updated solution file. (dotnet-architecture#1514)

Co-authored-by: ansonzhang <[email protected]>
Co-authored-by: m-knet <[email protected]>
Co-authored-by: Manuel Cañete <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>
@thanhauco
Copy link

Hello @sughosneo @anjoy8,

I am a newbie in this repo, I'd like to contribute to this project, is there a bug list from easy to hard for me to get familiar with project?

Thanks,
Thanh

@anjoy8
Copy link
Contributor Author

anjoy8 commented Nov 22, 2020

Hello @sughosneo @anjoy8,

I am a newbie in this repo, I'd like to contribute to this project, is there a bug list from easy to hard for me to get familiar with project?

Thanks,
Thanh

@thanhauco Currently there is no list, but you can modify the bug from the Issue list

@sughosneo
Copy link
Contributor

Hello @sughosneo @anjoy8,
I am a newbie in this repo, I'd like to contribute to this project, is there a bug list from easy to hard for me to get familiar with project?
Thanks,
Thanh

@thanhauco Currently there is no list, but you can modify the bug from the Issue list

Thanks @anjoy8 . That's correct.

Thank you for your interest @thanhauco . You could pick up any open item of your choice and submit a PR against that. We will be happy to review and merge the changes.

nishanil added a commit that referenced this issue Jan 20, 2021
* Fix functional tests of Marketing

* remove mvc routing spa

* move background task

* restore background config

* background

* Add health check in Ordering.BackgroundTask project

* add package version to dependencies.props

* ordering.api error integrationevents

* remove not needed code

* Fix authentication in Ordering.SignalR

* Create ReadMe.md

* partial checkin trying to fix checkout process

* add grpc basket

* first approach

* restore options and grpc calling from console app it's working

* Partial checkin

* fix grpc call from mobile bff to basket.api and add ordering, in ordering controller

* migrate web bff to 3.0 and apply grcp in order controller

* Add proto to catalog

* Moved some files to ./src/

* Moved .env to src

* partial checkin

* web bff endpoint /api/v1/Basket working in grpc

* refactoring the grpc client

* refactored the grpc client

* Update Dockerfiles for use /src as build context root

* refactor mobile bff

* Deleted unneeded Dockerfile

* update dockerfiles

* Fix mvc calls to grpc

* temporal fix to get the order in local dbset of orders for UpdateOrderWhenBuyerAndPaymentMethodVerifiedDomainEventHandler.cs

* add using

* fix checkout process of mvc app

* Migrate packages in preview7 to preview8

* fix identity server

* standarize the productId type

* format

* Add missing key in appsettings

* fix version problems with identity server

* Add layers in docker for npm install

* fix problems with spa adding items to basket

* add polly for grpc calls

* Fix identity server

* fix tests of basket

* fix unit test of basket

* fix ordering tests

* updated compose and 1st envoy config

* updated config files

* update to helm charts for grpc config

* fixed ending 0 in values.yaml for catalog

* Final charts tunning and allowing Linkerd Mesh to be used

* update to net3 rc1

* Configuration of envoy routes and minor code fixes

* missing word in comment :p

* updated charts for local k8s deployment

* Fixing some generic issues

* Added web shoping agg to mesh

* ApiGW added to mesh. Ingress updated for Linkerd

* Final adjustments to make envoy work with ingress and remove Polly from http calls

* Removing unneeded files. Making compatible FailingMiddleware & Healthchecks

* Changes in compose config for docker-compose up running ok. Added volumes for data

* Update dockerfile.develop to net3

* Readme updated to net3

* fixed path in azure devops build

* Sort package references

* Aling basket-data-test entry with base .yml

* Remove unnecessary commands from dockerfiles

* Add back logging traces referenced in the wiki

* Remove string interpolation from logging

* Change obsolete .ForSqlServerUseSequenceHiLo for .UseHiLo

* Remove obsolete warning configuration

* Replace obsolete IHostingEnvironment for IWebHostEnvironment

* Remove obsolete references

* Sort dependencies.props

* Removed obsolete folders

* Fix webstatus configuration for docker-compose

* Update README.md

* MVC working with SignalR without Envoy

* Remove SignalR transport parameter on clients

* Remove unsupported ServiceFabric folder

* Remove obsolete folders

* Reorganize project folders

* Improve docker build cache usage

* Add IdentityUrlExternal config in appsettings

* Fix tests compilation errors

* Run unit and integration tests from VS

* Add required configuration

* Fix to close issue #1162

* Add configuration value for running in Mac

* Relocate/remove folders

* Remove old test docs already in wiki

* Move more obsolete files

* Move auxiliary solution files to "to-fix" folder

* Remove obsolete files/folders

* Relocate mobile solutions, remove obsolete ones.

* Rename script to create aks resource

* Add comment to Create-DockerfileSolutionRestore.ps1 script

* Remove deleted folder from .gitignore

* Remove obsolete files

* Make installation work in local kubernetes

* Update README

* Move/remove obsolete folders

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Streamline README, add links to Wiki

* Minor README updates

* Bold release notes

* Fix SignalR + MVC through Envoy

* Change dockerhub microsoft images to mcr

* Fix services test containers

* Fix SignalR + SPA through Envoy

* Fix paths for moved folders

* Comment out windows jobs

* Fix healthchecks configuration values

* Configure Envoy to forward SignalR notifications

* Unify dashboard setup yaml

* Simplify ingress setup files

* Explore adding CSP

* Update to 3.1
- Fix test to work on Catalog.API controller, using in-memory database

* Don't force LongPolling for SignalR

* Update base images to 3.1

* Upgrade Locations to 3.1

* Relocate network-related scripts

* tls support

* Update packages

* Add missing await

* Update to IHost
- Return IHost from CreateHostBuilder
- Register AutoFac with UseServiceProviderFactory() in CreateHostBuilder

* Add back looging to Seq and Logstash

* Fix services names in azure-pipelines

* Update README.md

* fixed #1181

* Fix npm & docker build warnings #1224
Update Angular 7 to 8 to fix warnings and vulnerabilities

* Diferentiate displayNames

* Add project-directory param for compose

* Add -f parameter

* Add cache use comment to Dockerfiles to force PR validation

* minor change to requeue DevOps validation

* User internal docker dns names

* Update README.md

* Error fixed, applied the cookie policy to lax mode

* disable windows containers in buildimages and multiarch yaml (#1287)

* disable windows containers in buildimages and multiarch yaml

* fix space

* disabled delopy in windows and prefixes

* Change extension method to load the configuration

* fix authorization in webhooks api

* Remove obsolete middleware UseApplicationInsights

* Fix/include pipeline yml in trigger (#1290)

* add trigger to include pipelines

* Update build/azure-devops/webhooks-client/azure-pipelines.yml

Co-Authored-By: Miguel Veloso <[email protected]>

Co-authored-by: Miguel Veloso <[email protected]>

* Executed npm audit fix and npm update

* Executed npm audit fix

* correct call order of "app.UseCors"

* Updated readme with new book cover icon

* Bump npm-registry-fetch from 4.0.2 to 4.0.5 in /src/Web/WebSPA (#1361)

Bumps [npm-registry-fetch](https://github.com/npm/registry-fetch) from 4.0.2 to 4.0.5.
- [Release notes](https://github.com/npm/registry-fetch/releases)
- [Changelog](https://github.com/npm/npm-registry-fetch/blob/latest/CHANGELOG.md)
- [Commits](https://github.com/npm/registry-fetch/commits)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump lodash from 4.17.15 to 4.17.19 in /src/Web/WebSPA (#1365)

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.17.15...4.17.19)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump jquery from 3.4.1 to 3.5.0 in /src/Web/WebSPA (#1313)

Bumps [jquery](https://github.com/jquery/jquery) from 3.4.1 to 3.5.0.
- [Release notes](https://github.com/jquery/jquery/releases)
- [Commits](jquery/jquery@3.4.1...3.5.0)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump websocket-extensions from 0.1.3 to 0.1.4 in /src/Web/WebSPA (#1336)

Bumps [websocket-extensions](https://github.com/faye/websocket-extensions-node) from 0.1.3 to 0.1.4.
- [Release notes](https://github.com/faye/websocket-extensions-node/releases)
- [Changelog](https://github.com/faye/websocket-extensions-node/blob/master/CHANGELOG.md)
- [Commits](faye/websocket-extensions-node@0.1.3...0.1.4)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Updated readme.md with mslearn link

* Moved mslearn section on top

* Fixed typo in README (#1375)

* Shifting MS learn section on top.

* Fix/1370 - Updated script related changes for helm 3.x and k8s 1.16.x (#1378)

* Fix for helm 3 and k8s 1.16

* Inclusion of archived directory under k8s

* separate deploy-all powershell script for local Mac OS deployment.

* Bump elliptic from 6.4.1 to 6.5.3 in /src/Web/WebSPA (#1381)

Bumps [elliptic](https://github.com/indutny/elliptic) from 6.4.1 to 6.5.3.
- [Release notes](https://github.com/indutny/elliptic/releases)
- [Commits](indutny/elliptic@v6.4.1...v6.5.3)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Swamy/remove unused using and refactoring entity classes (#1371)

* Removed Unused Usings

* Code Refactoring for readbility. Added spaces and moved the classes to separate file

* Removed unused usings from ModelDTOs (#1373)

* Updated dot-prop version to 5.1.1 (#1382)

* Update README.md

* Update README.md

* Updated Readme.

* Fixed firewall rules check and improved the script

the check shall be like ~ Get-NetFirewallRule -DisplayName eShopOnContainers-* -ErrorAction Stop

* Updated serialize-javascript version to 3.1.0 (#1396)

* Update "minimist" version to 1.2.5 (#1398)

* #1397: Replaced deprecated docker.for.win.localhost by host.docker.internal in src/.env (#1400)

* Updated Readme (#1402)

Fixed sentence structure in Readme.

Changed "and a several" to "with several."

* CatalogService: Fix issue with Status set when items list is empty (#1304)

* Fix issue with Status set when items list is empty

* Change method Count() call to Count property

Co-authored-by: Dmytro Hridin <[email protected]>

* refactored Equals() method on ValueObject (#1316)

* Fix/1403and1404 removed duplicate Key SubscriptionClientName and added app.UseAuthorization() call (#1406)

* #1403 removed duplicate Key SubscriptionClientName

Removed duplicate key SubscriptionClientName from Tests/Services/Application.FunctionalTests/Services/Marketing/appsettings.json and sorted its content in asc order.

* #1404 Added app.UseAuthorization() call

Added app.UseAuthorization() call to BasketTestsStartup, LocationsTestsStartup, and MarketingTestsStartup to fix failed unit tests IntegrationEventsScenarios.Post_update_product_price_and_catalog_and_basket_list_modified and MarketingScenarios.Set_new_user_location_and_get_location_campaign_by_user_id (see #1404)

* Fix for Campaigns exception and SignalR 401 Unauthorized (#1374)

* update API Gateway - /locations-api/ @ webmarketing/envoy.yaml

* updated signalr services

- envoy: webmarketingapigw
- latest client: webmvc
- service hub: ordering-signalrhub

Co-authored-by: hfz-r <[email protected]>

* Mis-Spelled 'client' (#1411)

* fix parameter error in multiarch job (#1413)

* Private readonly string changed to private const string (#1288)

* fix disposing of direct instantiated objects in calalog service #1392 (#1395)

* Updated version of different packages. (#1420)

* for issue #1423: changed literal string "OpenIdConnect" to constant string (#1424)

Co-authored-by: Jeremiah Flaga <[email protected]>

* Updated node-fetch package version. (#1426)

* Updated node-fetch package version.

* Updated node-forge version.

* Fixes #1474: webspa container does not build when running docker-compose up.Updated sha hashes in packages-lock.json (#1475)

* Change ReadAllBytes to ReadAllBytesAsync in PicController (#1425)

* Fixed parameter bug create resource script (#1494)

* Fixed invalid number format in webspa (#1496)

* Updated package version of webpack-subresource-integrity (#1497)

* Fixed #1495 bug : UseLoadTest missing claimsType (#1508)

* Update ByPassAuthMiddleware.cs

* Update ByPassAuthMiddleware.cs

* remove WebMVC/ByPassAuthMiddleware duplicate cliam

* remove Ordering.API/ByPassAuthMid  duplicate cliam

* remove Location.API/ByPassAuthMid  duplicate cliam

* remove Market.API/ByPassAuthMid  duplicate cliam

* change webmvc bypassauth: claimType of sub

* change ordering bypassauth: claimType of sub

* change location bypassauth: claimType of sub

* change market bypassauth: claimType of sub

* Update solution items (#1515)

Co-authored-by: Manuel Cañete <[email protected]>

* Updated solution file. (#1514)

* Fixed #1518 bug (#1520)

* Update CatalogContextModelSnapshot.cs

* Update CatalogContextSeed.cs

* Modify CSV data

* Give PictureFileName assignment

* Fix PurchaseUrl port in WebSPA appsettings.json (#1525)

* Updated highlight version to latest. (#1531)

* Finished sentence and fixed spelling mistake. (#1536)

* Updated credit card default exipiration value. (#1535)

* Fix login issue Exception: Correlation failed (#1516)

Fix samesite issue when running eShop from docker-compose locally as by default http protocol is being used
Refer to #1391

* Updated getting started section in readme file. (#1538)

* Updated get started section in readme file.

* Updated docker-compose specific link

* Updated instructions in the readme file.

* Updated packages to latest version. (#1539)

* Update README.md

* fix(#1261): remove helm chart indent for config map glob (#1533)

* Updated ini package in package-lock.json file. (#1541)

* Removed unnecessary await (#1546)

* 1. remove unused using, 2. Reorganized the using, 3. Moved the class into separate file and 4. Added line and spacing formats (#1532)

* Removed Unused Using and Reorganized the Using

* Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API

* Revert "Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API"

This reverts commit 34241c4.

* Removed unused using and reorganized the using inside "Services" folder

* Removed Unused using and reoganized the using

* Refactor Webhooks.API

* Removed unused using and reorganized using inside Catalog.API

* Swamy/remove unused using and refactor the code (#1552)

* Removed Unused Using and Reorganized the Using

* Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API

* Revert "Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API"

This reverts commit 34241c4.

* Removed unused using and reorganized the using inside "Services" folder

* Removed Unused using and reoganized the using

* Refactor Webhooks.API

* Removed unused using and reorganized using inside Catalog.API

* Refactoring

* Removed unsed using

* Added line break just to differentiate between the messages

* Swamy/remove unused using and refactor the code (#1555)

* Removed Unused Using and Reorganized the Using

* Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API

* Revert "Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API"

This reverts commit 34241c4.

* Removed unused using and reorganized the using inside "Services" folder

* Removed Unused using and reoganized the using

* Refactor Webhooks.API

* Removed unused using and reorganized using inside Catalog.API

* Refactoring

* Removed unsed using

* Added line break just to differentiate between the messages

* Removed unused usings

* Simple Refactoring

* Fixed Issue #1410 (#1560)

Added IdentityUrlExternal to src\Services\Ordering\Ordering.FunctionalTests\appsettings.json; sorted the properties

* Small Refactoring (#1558)

* Small-Formatting-Refactoring (#1564)

* Swamy/01jan2021-small-refactoring (#1570)

* Update Startup.cs

* Removed commented code

* Update UrlsConfig.cs

* Small Refactoring

* Removed Commented Code

* Small Refactoring

* Removes dotnet.myget.org NuGetPackage feed references (#1576)

* Removed Nuget package feed

* Removed unused System.ValueTuple package

* Downloadgraded Microsoft.AspNetCore.Hosting.Abstractions package

* Removed unused package

* Downgraded System.IO.Compression.ZipFile package

* Used Microsoft.AspNetCore.DataProtection.StackExchangeRedis package

* Swamy/04 jan2021-small-refactoring (#1571)

* Small Refactoring inside IntegrationEventLogEF

* Small Refactoring inside BuildingBlocks folder

* Fixes #1581: Fixed total decimal place issue and User hover menu display/usability issue. (#1582)

* Swamy/15jan2021 small refactoring (#1580)

* Small Refactoring inside Basket Service folder

* Small Refactoring

* Incorrect ContentPage.ToolbarItems in a TabbedPage (#1348)

Incorrect ContentPage.ToolbarItems in a TabbedPage

Co-authored-by: Erik Pique <[email protected]>
Co-authored-by: ericuss <[email protected]>
Co-authored-by: eiximenis <[email protected]>
Co-authored-by: Miguel Veloso <[email protected]>
Co-authored-by: Miguel Veloso <[email protected]>
Co-authored-by: ericuss <[email protected]>
Co-authored-by: Christian Morgenthaler <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sander Holvoet <[email protected]>
Co-authored-by: Sumit Ghosh <[email protected]>
Co-authored-by: Viswanatha Swamy <[email protected]>
Co-authored-by: Mark Harwood <[email protected]>
Co-authored-by: edmondshtogu <[email protected]>
Co-authored-by: InstanceFactory <[email protected]>
Co-authored-by: Yosef Herskovitz <[email protected]>
Co-authored-by: Dmytro Hridin <[email protected]>
Co-authored-by: Dmytro Hridin <[email protected]>
Co-authored-by: André Silva <[email protected]>
Co-authored-by: hfz-r <[email protected]>
Co-authored-by: hfz-r <[email protected]>
Co-authored-by: Majid Ali Khan Quaid <[email protected]>
Co-authored-by: Javier Vela <[email protected]>
Co-authored-by: Facundo La Rocca <[email protected]>
Co-authored-by: Nabil Sedoud <[email protected]>
Co-authored-by: jeremiahflaga <[email protected]>
Co-authored-by: Jeremiah Flaga <[email protected]>
Co-authored-by: Wojciech Rak <[email protected]>
Co-authored-by: Zakaria <[email protected]>
Co-authored-by: ansonzhang <[email protected]>
Co-authored-by: m-knet <[email protected]>
Co-authored-by: Manuel Cañete <[email protected]>
Co-authored-by: n-stefan <[email protected]>
Co-authored-by: Gerard Gunnewijk <[email protected]>
Co-authored-by: Yingting Huang <[email protected]>
Co-authored-by: Mads Rosenberg <[email protected]>
Co-authored-by: Gur Galler <[email protected]>
Co-authored-by: Hetal Kapadia <[email protected]>
Co-authored-by: william-keller <[email protected]>
jyeros pushed a commit to jyeros/eShopOnContainers that referenced this issue Aug 15, 2023
…otnet-architecture#1508)

* Update ByPassAuthMiddleware.cs

* Update ByPassAuthMiddleware.cs

* remove WebMVC/ByPassAuthMiddleware duplicate cliam

* remove Ordering.API/ByPassAuthMid  duplicate cliam

* remove Location.API/ByPassAuthMid  duplicate cliam

* remove Market.API/ByPassAuthMid  duplicate cliam

* change webmvc bypassauth: claimType of sub

* change ordering bypassauth: claimType of sub

* change location bypassauth: claimType of sub

* change market bypassauth: claimType of sub
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants