-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Description
Location: UrlDetector.scala:42
AbsoluteUrl.parse() is used inside a .map, which can throw exceptions on invalid URLs. This violates our FP/no-throw guidelines and can cause unexpected crashes:
- Function is not total (can fail unexpectedly)
- Breaks referential transparency
- Uncaught exceptions in
.map()will crash the application
To-Do
- Replace any usage of AbsoluteUrl.parse(...) (which can throw exceptions) with a safe alternative
- Add tests for invalid URLs
Metadata
Metadata
Assignees
Labels
No labels