Skip to content
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

Run development web server in insecure mode #2625

Merged
merged 1 commit into from May 25, 2023

Conversation

lunkwill42
Copy link
Member

The development web container doesn't work properly if DJANGO_DEBUG=False is set in nav.conf, since the staticfiles app would not serve most static resources in that case. This would prevent us from seeing what a non-debug site looks like while developing.

@lunkwill42 lunkwill42 self-assigned this May 25, 2023
Copy link
Contributor

@podliashanyk podliashanyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

Copy link
Contributor

@hmpf hmpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's also using whitenoise to always serve the static files the same way in prod, dev and whatever but this works too.

@github-actions
Copy link

github-actions bot commented May 25, 2023

Test results

     12 files       12 suites   12m 36s ⏱️
3 225 tests 3 129 ✔️   96 💤 0
9 150 runs  8 862 ✔️ 288 💤 0

Results for commit 608d9b3.

♻️ This comment has been updated with latest results.

The development web container doesn't work properly if
 `DJANGO_DEBUG=False` is set in `nav.conf`, since the staticfiles app
 would not serve most static resources in that case.  This would
 prevent us from seeing what a non-debug site looks like while
 developing.
@sonarcloud
Copy link

sonarcloud bot commented May 25, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@lunkwill42
Copy link
Member Author

There's also using whitenoise to always serve the static files the same way in prod, dev and whatever but this works too.

Yup, whitenoise is great, but won't we have to run the collectstatic command every time we make a change, @hmpf ? (can't remember, it's been a while since I used whitenoise)

@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Merging #2625 (608d9b3) into 5.6.x (5b394ee) will increase coverage by 53.99%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           5.6.x    #2625       +/-   ##
==========================================
+ Coverage       0   53.99%   +53.99%     
==========================================
  Files          0      558      +558     
  Lines          0    40621    +40621     
==========================================
+ Hits           0    21934    +21934     
- Misses         0    18687    +18687     

see 558 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lunkwill42 lunkwill42 merged commit 6b632f3 into Uninett:5.6.x May 25, 2023
12 checks passed
@lunkwill42 lunkwill42 deleted the devfix/runserver-insecure branch May 25, 2023 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants