feat: Implement upload/download of trace records#22
Merged
alehander92 merged 34 commits intomainfrom Mar 17, 2025
Merged
Conversation
pxor
force-pushed
the
feat-upload-download-traces
branch
3 times, most recently
from
f838285 to
b8a295a
Compare
Madman10K
suggested changes
Mar 11, 2025
Contributor
Madman10K
left a comment
Madman10K
left a comment
There was a problem hiding this comment.
Looks good to me, just needs to be changed to follow the common code style. Maybe @alehander92 can add to the review
alehander92
reviewed
Mar 11, 2025
alehander92
reviewed
Mar 13, 2025
pxor
changed the title
alehander92
reviewed
Mar 13, 2025
pxor
force-pushed
the
feat-upload-download-traces
branch
2 times, most recently
from
a0bfe93 to
d62665f
Compare
Contributor
Author
|
|
Contributor
Author
|
|
alehander92
reviewed
Mar 14, 2025
alehander92
suggested changes
Mar 14, 2025
Contributor
alehander92
left a comment
alehander92
left a comment
There was a problem hiding this comment.
looking good, a few changes left!
pxor
force-pushed
the
feat-upload-download-traces
branch
2 times, most recently
from
3970d59 to
22744e3
Compare
Currently there is a GUI for the upload button When clicked, it zips the given trace with a random password encryption and uploads it to the provided web api root
…ime to the database file
In the nim-lan/zip library password encryption is not supported so we are currently using the standard zip and unzip packages
Still need to encrypt the file and set up the codetracer web api
…ugh <downloadId>::<password>
…ding on the trace fields Add fields for the downloadKey, controlId and expireTime to the trace object Add a Command for delete of trace files Handle deleting of the trace file's download key, controlId and expireTime
…nished and handle the frontend update of the trace object
… tooltip, when download key has been copied
…ire time and change the text to an icon
…oll all of the available recent traces
…get upload/delete/download links for ease of use
…oad <downloadKey>`
pxor
force-pushed
the
feat-upload-download-traces
branch
from
22744e3 to
664eb69
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
internal support for trace sharing, graphical interface for them and implementing underlying
ct download <download-key>/ct uploadcommands.some screenshots:
recent traces:
graphical interface for opening
downloadlinks