Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

WIP: Added option to save files in archive #564

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MrCyjaneK
Copy link
Contributor

@MrCyjaneK MrCyjaneK commented Sep 24, 2021

This should close #485, which I've opened a couple of months ago.

Usage is simple, just add:

archive:
 - test.txt
 - file2.txt

to your .abstruse.yml and files will be saved in ~/data/archive/{job id}.

Currently there is no way to serve the files with the api (code is written, it just doesn't work, probably an issue with chi?).

TODO:

  • Save files on server
  • Provide a simple ui for browsing the archived files
  • Redirect from /archive/{repo name}/{branch/pull request id} to /archive/{job id}

router.Mount("/api/v1", r.apiRouter())

router.Get("/archive/{id}/", build.HandleArchive(r.Jobs, r.Config))
Copy link
Contributor Author

@MrCyjaneK MrCyjaneK Sep 24, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need extra help with this line - it doesn't work in a way in which I'd expect it to work... I get a 404 instead of the response from function, when requesting /archive/{id}/file.txt

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Save file after passing a test
1 participant