-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add error logs #66
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does core.error support two args though?
It's really weird that this gets logged with accurate data but then it failed. Maybe you can try with a token and replicate the fetch locally?
it does support 2 args, but you need to provide the right fields for the 2nd https://github.com/actions/toolkit/blob/main/packages/core/src/core.ts#L250-L259 |
updated |
Probably issue is with token access. that what i want to know with error in logs |
Can we get this in, Asana linking is completely broken for us |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Adding the logs is fine but might be better to get a token to debug this
Might be necessary to introduce a retry logic here for instance. Could be ETIMEDOUT erorrs |
@lanre-mark @federico-po it shows "Error while completing task(s) (1207629198924482). Error: Forbidden" now |
No issues when trying to complete task (using https://developers.asana.com/reference/updatetask and personal token). @federico-po how to check what token is used for CI , maybe it doesn't have access? |
maybe that's it - you should ask the repo owner (or someone with enough access to it). The token needs to be in the repo config under |
@farisissa8 do you know who is mobile repo owner? |
Summary
can't understand why asana complete task action fails. add logs to this an other places.