Skip to content

Redirect deprecated SSE to Web IDE

Chad Woolley requested to merge caw-reroute-sse into master

What does this MR do and why?

Redirects deprecated Static Site Editor (SSE) to Web IDE.

See #349729 (closed) for details

Note it does not remove any deprecated SSE code other than the feature spec, that will be done in a follow-on issue/MR: #352505 (closed)

How to set up and validate locally

  1. Create a project in GDK, e.g. sse-testing, with a repo and README.md file.
  2. Go to the SSE route, e.g.: http://gdk.test:3000/testgroup/sse-testing/-/sse/main%2FREADME.md for dev, or https://gitlab.com/gitlab-com/www-gitlab-com/-/sse/master%2FREADME.md for production.
  3. See that the file is opened in the Web IDE instead of the SSE.

Example log entry from redirect

2022-05-21_23:48:09.13165 gitlab-workhorse:

{"content_type":"text/html; charset=utf-8","correlation_id":"01G3MGAV9226VNGCSXQ1PHCXKQ","duration_ms":520,"host":"gdk.test:3000","level":"info","method":"GET","msg":"access","proto":"HTTP/1.1","referrer":"","remote_addr":"127.0.0.1:56131","remote_ip":"127.0.0.1","route":"","status":302,"system":"http","time":"2022-05-21T16:48:09-07:00","ttfb_ms":520,"uri":"/testgroup/sse-testing/-/sse/main%2FREADME.md","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/101.0.4951.64 Safari/537.36","written_bytes":144}

So, the uri field will contain -/sse.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Deployment

This should be deployed at the beginning of %14.9

Edited by Chad Woolley

Merge request reports

Loading