Skip to content

Resolves #1739 and #1740, getOrg() Error Handling and Documentation#1778

Merged
david-rocca merged 2 commits intov2.8.0_featurefrom
af-1739
Apr 29, 2026
Merged

Resolves #1739 and #1740, getOrg() Error Handling and Documentation#1778
david-rocca merged 2 commits intov2.8.0_featurefrom
af-1739

Conversation

@afoote-mitre
Copy link
Copy Markdown
Collaborator

Closes Issue #1739 and #1740

Summary

Updated the getOrg() error handling to properly separate errors thrown by the repository, from errors through by mongo, and errors thrown when no results are found. Updated inline comments to properly reflect what each case handles.

Important Changes

src/controller/org.controller/org.controller.js

  • Added new error handling for different errors
  • Clarified accompanying documentation

Testing

Steps to manually test updated functionality, if possible

  • 1) Search for an org that doesn't exist
  • 2) Verify that a 404 returns as expected
  • 3) Stop local mongo instance
  • 4) Search for an org
  • 5) Verify that a 500 returns as expected

@afoote-mitre afoote-mitre self-assigned this Apr 23, 2026
@david-rocca david-rocca changed the base branch from v2.7.6_feature to v2.8.0_feature April 29, 2026 17:09
@david-rocca david-rocca merged commit dd12278 into v2.8.0_feature Apr 29, 2026
8 of 9 checks passed
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