Skip to content

Comments

Avoid NPE when bad queryKey is provided#252

Merged
labkey-adam merged 2 commits intodevelopfrom
fb_pepdb_npe
Feb 18, 2026
Merged

Avoid NPE when bad queryKey is provided#252
labkey-adam merged 2 commits intodevelopfrom
fb_pepdb_npe

Conversation

@labkey-adam
Copy link
Contributor

Rationale

The crawler induced an NPE in pepdb-getPeptides.view by supplying a bogus queryKey parameter value

Changes

  • Convert a couple chains of if statements into switch (thanks, Claude)
  • Show the search form and an error message if queryKey is unrecognized. This matches the queryKey missing case.

@labkey-adam labkey-adam merged commit dba992a into develop Feb 18, 2026
7 of 8 checks passed
@labkey-adam labkey-adam deleted the fb_pepdb_npe branch February 18, 2026 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants