dotnet/aspnetcore

NavigationException doesn't have a useful message.

Open

#51.787 aperta il 31 ott 2023

Vedi su GitHub
 (10 commenti) (0 reazioni) (0 assegnatari)C# (10.653 fork)batch import
area-blazorenhancementfeature-blazor-navigationhelp wanted

Metriche repository

Star
 (37.933 star)
Metriche merge PR
 (Merge medio 16g 9h) (258 PR mergiate in 30 g)

Descrizione

this https://github.com/dotnet/aspnetcore/blob/d8a3aa870862d01a8b3a3235f8dd7fc951dd8ff4/src/Components/Server/src/Circuits/RemoteNavigationManager.cs#L91

creates an exception like this image with the URL in the Location field.

Ideally it should have a useful message that includes the URI, eg., "Could not navigate to https://localhost:7298/item/99"

the problem is that NavigationException is not overriding Message and ToString https://github.com/dotnet/aspnetcore/blob/d8a3aa870862d01a8b3a3235f8dd7fc951dd8ff4/src/Components/Components/src/NavigationException.cs#L9

If we want to keep the URL out of the message for security reasons, it should at least have a message like "Could not navigate to the URL".

Guida contributor

Direzione di ricerca
La correzione prevede di sovrascrivere la proprietà Message in NavigationException (src/Components/Components/src/NavigationException.cs) per includere l'URI dal campo Location. Facoltativamente, considerare le preoccupazioni di sicurezza riguardanti l'esposizione dell'URL. L'eccezione attuale viene creata in RemoteNavigationManager.cs (src/Components/Server/src/Circuits/RemoteNavigationManager.cs). Rivedere la classe di eccezione esistente e assicurare una formattazione corretta.
Tech stack
csharp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80