-
Notifications
You must be signed in to change notification settings - Fork 46
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
Trace hosting server #52
Comments
Great start! I think the response should return a URL to the raw trace and a link to view it:
Seems reasonable. I could even see this being less, but this is mostly a cost issue. |
SGTM! i dig it. |
@tollmanz do you mean that Hence sticking to your case, response will be kinda
|
@denar90 That makes a lot of sense. This is hosted on Github and would not make sense to proxy through here. Going direct to the Firebase location makes a ton of sense. |
@denar90 yes i think your interpretation is mostly correct. The limitation we're talking about is that we can't run any server on {
raw: 'https://traces.firebaseio.com/traces/CuriousRedCrow',
view: 'https://chromedevtools.github.io/timeline-viewer/?loadTimelineFromURL=https://traces.firebaseio.com/traces/CuriousRedCrow'
} Currently I think we shouldn't abbreviate the URL (like the |
sgtm 👍 |
ya. later, we could extend it so the user can choose: A) save permanently in your gdrive or B) save here for 90days |
happy to help where I can :) |
attaching repo to not lose or forget about it |
https://chromedevtools.github.io/timeline-viewer/ has the ability to load traces over XHR from any host that's providing the JSON with proper CORS headers. (The google drive integration is designed to set the right permission and use the undocumented (lol) URL that is served with CORS headers.
Ideally though, folks would be able to POST a trace to a public endpoint. That would, in turn, provide a URL where the trace is now available. And then it's just a matter of opening
https://chromedevtools.github.io/timeline-viewer/?loadTimelineFromURL=<hosted trace url>
I'd be happy to foot the bill for this for now, and I feel confident I can secure a long-term billing situation.
Strawman proposal:
User flow:
/trace/upload
{"traceEvents":[
)/traces/<UUID>
I'm open to all ideas on the above. :)
The text was updated successfully, but these errors were encountered: