ghidra-lx-loader icon indicating copy to clipboard operation
ghidra-lx-loader copied to clipboard

CI: Avoid api rate limiting

Open antoniovazquezblanco opened this issue 1 year ago • 4 comments

I was looking at the pipeline after the merge. The error you got is quite unusual. To avoid it you may use an auth token as in this PR or you may run the pipeline again in some time.

This seems to be happening because the same runner has been running many setup-ghidra actions today which is unfortunate.

antoniovazquezblanco avatar Jun 17 '24 17:06 antoniovazquezblanco

Yes, it's not problem with the code, as I have tested it in PR as well, and it builded correctly. I have tried to re run the build. Let's see what happens.

oshogbo avatar Jun 17 '24 17:06 oshogbo

Perfect! Please, feel free to either merge or close this PR. I understand that people may not want to be sharing their token with random pipeline actions.

Thanks!

antoniovazquezblanco avatar Jun 17 '24 17:06 antoniovazquezblanco

Looks like er28-0652/setup-ghidra supports @actions/tool-cachenpm package (https://github.com/actions/toolkit/tree/main/packages/tool-cache); maybe it should be possible to cache entire Ghidra installation?

https://github.com/actions/gh-actions-cache could be used to check if caches are saved

halamix2 avatar Aug 04 '24 18:08 halamix2

It is supported and already done with my setup action...

antoniovazquezblanco avatar Aug 05 '24 06:08 antoniovazquezblanco

@oshogbo should I close this PR?

antoniovazquezblanco avatar Aug 31 '24 11:08 antoniovazquezblanco

Yes. It seems to work now :)

oshogbo avatar Sep 02 '24 08:09 oshogbo