dotnet/aspnetcore

NavigationException doesn't have a useful message.

Open

#51.787 geöffnet am 31. Okt. 2023

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (10.653 Forks)batch import
area-blazorenhancementfeature-blazor-navigationhelp wanted

Repository-Metriken

Stars
 (37.933 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 16T 9h) (258 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Die Korrektur beinhaltet das Überschreiben der Message Eigenschaft in NavigationException (src/Components/Components/src/NavigationException.cs), um den URI aus dem Location Feld einzubeziehen. Optional sollten Sicherheitsbedenken bezüglich der Offenlegung der URL berücksichtigt werden. Die aktuelle Ausnahme wird in RemoteNavigationManager.cs (src/Components/Server/src/Circuits/RemoteNavigationManager.cs) erstellt. Überprüfen Sie die bestehende Ausnahmeklasse und stellen Sie eine korrekte Formatierung sicher.
Tech Stack
csharp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80