Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide more intuitive error message when JFrog feed is inactive #1427

Merged
merged 5 commits into from
Oct 4, 2023
Merged

Provide more intuitive error message when JFrog feed is inactive #1427

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
65d25b4
provide more intuitive error message when JFrog feed is inactive
anamnavi Sep 29, 2023
e165ad9
update RequiredResource test to have an empty inner hashtable scenario
anamnavi Sep 29, 2023
d29b038
Revert "update RequiredResource test to have an empty inner hashtable…
anamnavi Oct 3, 2023
ec6fe9a
catch specific exception
anamnavi Oct 4, 2023
8554746
catch specific exception
anamnavi Oct 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions src/code/V3ServerAPICalls.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1472,6 +1472,15 @@ private JsonElement[] GetJsonElementArr(string request, string propertyName, out
entries = responseEntries.ToArray();
}
}
catch (JsonException e)
{
// scenario where the feed is not active anymore, i.e confirmed for JFrogArtifactory. The default error message is not intuitive.
errRecord = new ErrorRecord(
exception: new Exception($"JSON response from repository {Repository.Name} could not be parsed, likely due to the feed being inactive or invalid, with inner exception: {e.Message}"),
"FindVersionGlobbingFailure",
ErrorCategory.InvalidResult,
this);
}
catch (Exception e)
{
errRecord = new ErrorRecord(
Expand Down
Loading