Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Remove server /timing route #2428

Closed
g-k opened this issue Mar 20, 2017 · 4 comments
Closed

Remove server /timing route #2428

g-k opened this issue Mar 20, 2017 · 4 comments
Labels
good first issue security Security issue: can be an active issue, or related to security hygene
Milestone

Comments

@g-k
Copy link
Contributor

g-k commented Mar 20, 2017

» curl -vX POST 'https://pageshot.net/timing'
*   Trying 35.162.69.87...
* Connected to pageshot.net (35.162.69.87) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
* Server certificate: pageshot.net
* Server certificate: DigiCert SHA2 Secure Server CA
* Server certificate: DigiCert Global Root CA
> POST /timing HTTP/1.1
> Host: pageshot.net
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 500 Internal Server Error
< Content-Security-Policy: default-src 'self'; img-src 'self' www.google-analytics.com pageshot-usercontent.net data:; script-src 'self' www.google-analytics.com 'nonce-e0dec415-b732-421f-8563-7a7893daef85'; style-src 'self' 'unsafe-inline' https://code.cdn.mozilla.net; connect-src 'self' www.google-analytics.com sentry.prod.mozaws.net; font-src https://code.cdn.mozilla.net;
< Content-Type: text/plain; charset=utf-8
< Date: Mon, 20 Mar 2017 15:15:23 GMT
< ETag: W/"c-cx4CTq1LG2ze3rjnQ3PEUQ"
< Strict-Transport-Security: max-age=86400000
< X-Frame-Options: DENY
< X-Powered-By: Express
< Content-Length: 12
< Connection: keep-alive
<
* Connection #0 to host pageshot.net left intact
Server error%  
@jvehent jvehent added the security Security issue: can be an active issue, or related to security hygene label Mar 20, 2017
@ghost
Copy link

ghost commented Mar 20, 2017

Let's remove /timing

@ghost ghost added this to the Stretch milestone Mar 20, 2017
@ghost ghost added the good first issue label Mar 20, 2017
@ianb
Copy link
Contributor

ianb commented Mar 20, 2017

Note the /timing route is in server/src/server.js

@ianb ianb changed the title POST /timing returns 500 Remove server /timing route Jun 28, 2017
@brainzest
Copy link
Contributor

@ianb : The code for this has to be deleted or commented out for now?

@g-k
Copy link
Contributor Author

g-k commented Jul 31, 2017

@brainzest delete it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue security Security issue: can be an active issue, or related to security hygene
Projects
None yet
Development

No branches or pull requests

4 participants