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

blog: security patches for Sep 24 #1607

Draft
wants to merge 16 commits into
base: gh-pages
Choose a base branch
from
Draft

Conversation

UlisesGascon
Copy link
Member

@UlisesGascon UlisesGascon commented Sep 11, 2024

This blog entry should go live on Sept 30 (Monday).

cc: @expressjs/security-wg @expressjs/security-triage

Copy link

netlify bot commented Sep 11, 2024

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 9f536e1
🔍 Latest deploy log https://app.netlify.com/sites/expressjscom-preview/deploys/66e82cfd0164e200080b3757
😎 Deploy Preview https://deploy-preview-1607--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@UlisesGascon
Copy link
Member Author

@chrisdel101 I was not able to find this entry in the preview. Maybe is because has a future date? 🤔

@UlisesGascon UlisesGascon self-assigned this Sep 11, 2024
@UlisesGascon
Copy link
Member Author

Hey @lirantal! I will love to get your feedback on the format if possible :)

@chrisdel101
Copy link
Contributor

Your right! I guess we didn't test this enough. I'm not sure why immediately but I'm looking into this!

@chrisdel101
Copy link
Contributor

So yes, it is because it's future dated. If you want to change the file name date to today to preview it, and then once it's done change it back? Just for now?
We'll need to think about how we want to handle this workflow.

Copy link

@rxmarbles rxmarbles left a comment

Choose a reason for hiding this comment

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

LGTM

_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
@@ -0,0 +1,115 @@
---
title: September 2024 Security Releases!
Copy link
Member

@bjohansebas bjohansebas Sep 15, 2024

Choose a reason for hiding this comment

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

Can you add the 'description' metadata? The page now supports that function.

Suggested change
title: September 2024 Security Releases!
title: September 2024 Security Releases!
description: <description>

Copy link

@lirantal lirantal left a comment

Choose a reason for hiding this comment

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

Ran a first pass on the doc.

_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
_posts/2024-09-30-security-releases.md Outdated Show resolved Hide resolved
Comment on lines +104 to +108
**Patched versions**
- `8.0.0`
- `1.9.0`
- `0.1.10`
- `3.3.0`

Choose a reason for hiding this comment

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

these versions should be fixed to provide appropriate version ranges between the different majors.

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.

6 participants