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

[BUG] After explicit logout, hit on login, then it is automatically recognized instead of showing the login page #626

Closed
BrunoLopes opened this issue Jun 5, 2018 · 4 comments

Comments

@BrunoLopes
Copy link

BrunoLopes commented Jun 5, 2018

Hi guys, I can't get login page after a forced logout, it's getting logging in automatically in a mvc client application with hybrid grant type.

@CESARDELATORRE
Copy link
Collaborator

Yeah, this is a known issue/bug we'll need to fix when possible. Thanks for the feedback.

@mvelosop mvelosop changed the title After explicit logout, hit on login, then it is automatically recognized instead of showing the login page [BUG] After explicit logout, hit on login, then it is automatically recognized instead of showing the login page Jun 7, 2018
@mvelosop mvelosop added the bug label Jun 7, 2018
@BrunoLopes
Copy link
Author

BrunoLopes commented Jun 8, 2018

I wish you guys prioritize this as soon as possible.

@Devs4life
Copy link

You should SignOut Identity on Identity.API
Ex: _signInManager.SignOutAsync();

@mvelosop
Copy link
Collaborator

mvelosop commented Feb 8, 2019

Closing this issue as the fix is confirmed.

@mvelosop mvelosop closed this as completed Feb 8, 2019
onixus74 added a commit to onixus74/eShopOnContainers that referenced this issue Apr 1, 2019
* Updated Microservices eBook for eReaders (.MOBI and .EPUB) to v2.1.02

* Migrate WebMVC app to bootstrap 4

* update swagger nuget to 2.4.0 from 3.0.0

* Removed bower from WebStatus app
Migrated WebStatus app to Bootstrap 4

* Migrated Identity.api to Bootstrap 4

* Fixed issue header not included in requests
Fixed issue httpclient returned response

* MediatorR: Migration Guide 4.x to 5.0

* Fixed broken UI in SPA after migrating to Bootstrap 4

* Delete eShopOnContainers.TestRunner.iOS.csproj.bak

* Fixed issue webmvc signalr not notifying submitting state

* ✔ Typo  fix  "suittable" => "suitable"

* Initial approach

* Updated OrderingIntegrationEvent svc

* Add private setters so deserializing on integration event handler works as expected.

* Send commands for modifying state in IntegrationEventHandlers

* [BUG] When subscribing more than 1 different event handlers for the same event type dotnet-architecture#645

dotnet-architecture#645

* [Bug] Deserialization of IntegrationEvent dotnet-architecture#667

dotnet-architecture#667

* Corrected the grammatical and spelling mistakes on Readme.md

* [BUG] After explicit logout, hit on login, then it is automatically recognized instead of showing the login page dotnet-architecture#626

dotnet-architecture#626

* Token lifetime handling dotnet-architecture#118

dotnet-architecture#118

* Send IntegrationEvents after committing transactions

* Refactoring IntegrationEventLog service

* Fix method name in OnDisconnectAsync

* Improves README

* Added Architecture and presentation decks

* Fixed typo.

* Fixed ordering unit test

* change netcore 2.1 to 2.2 and update nugets

* local k8s docker support

* Updated Ocelot version

* update dockerfiles

* add features api

* standar names and fixed response for swagger apis

* Refactoring Buyer aggregate

* add test to CI

* Fixed hardcoded redis connection in FunctionalTests

* get rid of flatmap-stream

* udpated npm-watch

* update test deploy

* Updated eBook (pdf) to v2.1.03

* Integrate HealthCheck to all eShop services and apps
Integrate WebStatus with Healthcheck

* Updated Helm scripts to include healthcheck url env vars
Updated Webstatus docker-compose to include HealthCheck url env vars

* Updated Webstatus startup
Ignore temp healthchecksdb

* Fixed rxjs version in package.json

* Updated tests

* Migrated to netcore 2.2

* Added serilog

* fix error on  aks deployment

* Fixed missing env variables in marketing.api in k8s scripts
Added Resource path in webstatus UI when deploying with K8S
Added Liveness healthChecks
Updated k8s healthcheck configuration

* fixed merge

* Fixed netcore2.2 CORS issue not allowing wildcard origins

* Updated ReadMe to include .NET 2.2 updates

* Updated eBook to 2.2

* Fixed type in HttpGlobalExceptionFilter

Changed meesage to message

* Fixed a minor Typo in the eBook.

* Update README.md

* Update README.md

* deleted global.json

* Webhooks API: WIP - Initial commit

* Publish eReader Formats

* Updated v2.2 book eReader formats

* WIP Webhooks

* WIP

* comment updated in compose

* healtchecks

* badge for buid status

* added port configuration for liveness/readiness probes in helm charts

* updated helm charts for final probes config on k8s

* webhooks API & client

* Add Serilog and Seq working in Docker

* Add integration event traces

* Remove MachineName from logger configuration, not really useful for eShopOnContainers

* Use structured logging for exceptions

* Add Seq as sink and some integration event traces

* Standarize log message

* new helm charts for webhooks

* Add Seq to Ordering.API

* Expore Program.AppName/AppShortName for logging

* Added traces for UserCheckoutAcceptedIntegrationEvent

* OrderController Recommendation

It's good idea to take advantage of GetOrderByIdQuery and handle by GetCustomerByIdQueryHandler

* Add log traces for integration event handling

* webhooks flow finished. Only missing bug in api that don't show the hooks

* Add logging to CreateOrderCommandHandler

* Remove specific version for Microsoft.AspNetCore.App

* Added ApiGateways to solution mimicking eShopOnContainers-ServicesAndWebApps solution structure

* AllowedCorsOrigins disabled for Xamarin client. Fixes dotnet-architecture#905

* webhooks finished

* forcing granturl to be in same origin as hook url

* typo

typo

* .gitignore site.*.js files, as the are created on opening the Project

* Rename BuyerEntityTYpeConfiguration.cs to BuyerEntityTypeConfiguration.cs

* Added Seq to Identity.API

* Added Seq to Locations.API

* Added Seq to Marketing.API

* Add Seq to Payment.API

* Add Seq to WebStatus

* Add Seq to ApiGatewayAggregators

* Ass Seq to SignalR.Hub

* Add LogContext to IntegrationEventHandlers

* Revert "Add Seq to ApiGatewayAggregators"

This reverts commit 21427eb.

* Adding logger to unit test

* Fix IntegrationEventIdContext logging property name

* Exclude generated files in solution .gtignore

* Add Seq to WebMVC

* Refactor LogContext for IntegrationEvent Handlers to include AppName

* Add log traces for commands

* Add LogContext for behaviour transacction

* Ensure transaction is committed in the correct context, when handling chained or nested commands

* Add Seq to ordering.backgroundtasks

* Add publishing integration events traces

* Fix naming inconsistency

* Include files rebuilt by bundling and minification upon opening VS

* Fix non-structured traces

* Update traces templates

* minor packages update

replace preview apis with production
replace EnableKubernetes with AddApplicationInsightsKubernetesEnricher
replace  loggerFactory.AddAzureWebAppDiagnostics(); with  loggerBuilder.AddAzureWebAppDiagnostics();

* replace  loggerFactory.AddConsole(); with  loggerBuilder.AddConsole();

* General refactoring for documentation

* Standarize ApplicationContext logging property

* runfix

* Fix css 404 on logging in

* Remove unused external logins classes

* Refactor ModelDTOs for a more consistent folder structure

* Add logging of subscription events

* Fix duplicate x-requestid headers

* Fix JSON structure error.

* Fix bad previous fix

* Restored original name

* Updated file names to match class and interface names.

* Rename orderStatusChangedToAwaitingValidationIntegrationEventHandler.cs to OrderStatusChangedToAwaitingValidationIntegrationEventHandler.cs

* Enabling istio on local kubernetes and first pod configuration, needs more work, unstable

* Enabling istio on local kubernetes and first pod configuration, needs more work, unstable

* Added support to istio with nginx-ingress

* Fix errors on las commit

* Added bash version of the Helm deploy script

* Build correct solution

* Fix path error in docker-compose command

* host don't need anymore

* Fix initialization of the use_custom_registry boolean

* Critical vulnerabilities solved

* Update JS packages to new versions without know vulnerabilities;
Add HealthChecks to appsettings so SPA can run from VS using services in containers

* Correctly set the backing field for the use-local-k8s parameter

* Tested on AKS

* Fixed changes on deploy-all.ps1 on AKS

* Fixed documentation errors

* Commented out local configuration, to avoid exception

* Split RabbitMQ channel create and consumer create.

Because when channel create the Subscribe is not register, so the previously stored message may lose. This fix split the channel create and consumer create, when Subscribe registered then call the consumer create function.

* Update HealthChecks UI libraries to properly handle missing services.

* scripts fot catalog

* updated build catalog

* First version with Kibana/ElasticSearch & Logstash, needs refactor

* updated build catalog 2

* new yaml files

* builds readme

* Update azure-pipelines.yml for Azure Pipelines

* Update azure-pipelines.yml for Azure Pipelines

* builds updated

* Finished

* updated builds to use git branch name

* Added Documentation

* Updated readme.md for builds info

* Corrected a Typo

* webhooks.client rebranded

* adding triggers to yaml builds

* Fixed NPM Issues and nuget packages

* Done Updating versions

* Update README add note for EventBus

* Updated pointing to ELK external service for testing and demo purposes

* Updated pointing to ELK external service for testing and demo purposes, documentation update

* Update EventBus notes

* Changed default logstashUrl to null

* Updated Bootstrap to 4.3.1 to fix security issues

* Fixed some security issues with npm audit

* infrastructure build

* Simplify Enumeration and CardType classes as proposed in PR dotnet-architecture#704

* Remove test class for MVC project

* removing uneeded files

* compose for put linux and win in tag

* PS1 for creating multiarch manifests

* fix markdown headers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants