[GitHub] csantanapr opened a new issue #124: log message should have trigger name on trigger not found in DB

[GitBox <gi...@apache.org>]

csantanapr opened a new issue #124: log message should have trigger name on trigger not found in DB
URL: https://github.com/apache/incubator-openwhisk-package-alarms/issues/124
 
 
   User reported on a lifecycle DELETE for `/whisk.system/alarms/once`
   He inspecting logs after the facts and see a few of these but logs don't have good info for debugging like the trigger name.
   
   ```
   "2018-01-21T17:09:13.091227454Z stdout: alarm: Error invoking whisk action: 404 { error: 'could not find the trigger in the database' }"
   ```
   This message is printed in getTrigger function which is called after verifying auth that user owns the trigger, so I think is sage to add the trigger name to the error message
   https://github.com/apache/incubator-openwhisk-package-alarms/blob/master/action/lib/Database.js#L79
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services