Details
Details
- Reviewers
dereckson - Maniphest Tasks
- T771: Allow to send notifications from the command line
- Commits
- rNPUSH9de15a097588: Add CLI notification functionality
Diff Detail
Diff Detail
- Repository
- rNPUSH notification-push
- Lint
No Lint Coverage - Unit
No Test Coverage - Branch
- master
- Build Status
Buildable 4372 Build 4639: arc lint + arc unit
Event Timeline
src/main.rs | ||
---|---|---|
55 | There is a last check we can do. If it worked, we receive a response with HTTP code 200. The reqwest response offers a status method to check if the code is 200/201/204: if !res.status().is_success() { eprintln!("The notifications server can't process the notification."); exit(2); // Let's use another code so a script can know at what step it failed. } We aren't especially interested by the specifics, as the notifications center also logged the error in its own log, and there is nothing the push script or the user using it can do. |