dotnet/aspnetcore

NavigationException doesn't have a useful message.

Ouverte

#51 787 ouverte le 31 oct. 2023

Voir sur GitHubGuide contributeur
 (10 commentaires) (0 réaction) (0 personne assignée)C# (10 653 forks)batch import
area-blazorenhancementfeature-blazor-navigationhelp wanted

Métriques du dépôt

Stars
 (37 933 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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".

Guide contributeur

Direction de recherche
La correction implique de remplacer la propriété Message dans NavigationException (src/Components/Components/src/NavigationException.cs) pour inclure l'URI du champ Location. Optionnellement, considérez les préoccupations de sécurité concernant l'exposition de l'URL. L'exception actuelle est créée dans RemoteNavigationManager.cs (src/Components/Server/src/Circuits/RemoteNavigationManager.cs). Examinez la classe d'exception existante et assurez une mise en forme correcte.
Stack technique
csharp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80