-
Notifications
You must be signed in to change notification settings - Fork 4
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
DT-840: Switch auth order round as not needed for static content #969
Conversation
d3f16c2
to
482fcde
Compare
@@ -20,6 +20,9 @@ RUN mkdir -p /app | |||
WORKDIR /app | |||
ADD . . | |||
|
|||
# Remove cypress as it takes ~1 minute to download and install | |||
RUN sed -ie 's/"cypress[^ ].*//g' /app/package.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied from use of force
backend/index.js
Outdated
app.use(routes(apis)) | ||
|
||
app.use(setupStaticContent({ app })) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move stuff that doesn't need a auth session above
@@ -2,7 +2,7 @@ version: '3.1' | |||
services: | |||
prisonstaffhub-wiremock: | |||
image: rodolpheche/wiremock | |||
container_name: prisonstaffhub | |||
container_name: prisonstaffhub-wiremock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename so doesn't conflict with prisonstaffhub
@@ -6,42 +6,23 @@ | |||
<meta name="viewport" content="width=device-width, initial-scale=1"> | |||
|
|||
<!--[if !(IE)]><!--> | |||
<link rel='stylesheet' href='<%= htmlWebpackPlugin.options.publicUrl %>/styles/grid.css' type='text/css'> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
never set so removed
@@ -50,12 +50,10 @@ | |||
"contentful": "^7.3.0", | |||
"cookie-session": "^2.0.0-beta.3", | |||
"csv-parse": "^4.4.6", | |||
"cypress-multi-reporters": "^1.4.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved into dev dependencies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
a526821
to
c9ded9e
Compare
backend/index.js
Outdated
@@ -31,18 +31,18 @@ app.set('view engine', 'njk') | |||
app.use(setupBodyParsers()) | |||
app.use(setupHealthChecks()) | |||
app.use(setupWebSecurity()) | |||
app.use(setupRedirects()) | |||
app.use(setupSass()) | |||
app.use(setupStaticContent({ app })) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This no longer takes app as a parameter now that the nunjuck setup has been pulled out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Revert "Revert "DT-840: Switch auth order round as not needed for static content (#969)" (#971)" This reverts commit 66c9741. * DT-840: Fix missing bundle.js when running in production mode * DT-840: Remove duplicate redirect * DT-840: Fix paths for static js content Co-authored-by: Jamie Draper <draper65@googlemail.com>
No description provided.