Project

General

Profile

Actions
Copy link

Bug #15893

closed

Fix "Non exhaustive pattern match" warning

Added by François ARMAND about 5 years ago. Updated about 5 years ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Architecture - Code maintenance
Target version:
5.0.14
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

Warning:(120, 28) match may not be exhaustive.
It would fail on the following input: Left((x: Object forSome x not in (bootstrap.liftweb.checks.NcfTechniqueUpgradeError, bootstrap.liftweb.checks.NcfTechniqueUpgradeError.FlagFileError, bootstrap.liftweb.checks.NcfTechniqueUpgradeError.NcfApiAuthFailed)))
        authResponse       <- tryo ( authRequest.asString , "An error occurred while authentication to ncf API", NcfApiAuthFailed)

The problem is with the tryo definition, where catcher is defined on throwable and we only catch NcfApiAuthFailed here.

Actions
Copy link

Also available in: Atom PDF