Skip to content

added some more information about the security process #2696

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

Merged
merged 1 commit into from
Jun 22, 2013

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Jun 4, 2013

No description provided.

@greggles
Copy link

greggles commented Jun 4, 2013

I see two deficiencies in the current policy:

  1. It doesn't state a day of the week to release issues. I have a strong preference for Symfony to release on Tuesdays. That gives downstream projects enough time in the week to package and release their code before it gets to the weekend. This is especially an issue for folks in Asia-Pac.
  2. It doesn't state a minimum amount of time to wait for coordination among the projects. I suggest 2 weeks as a minimum time unless an issue is actively being exploited.

I reviewed the ezPublish security releases from 2012:

  • 6 dec 2012 - first thursday
  • 13 sep 2012 - second thursday
  • 8 july 2012 - first sunday
  • 9 may 2012 - first wednesday
  • 26 march - fourth monday

I don't know the details of those and whether there were active exploits in the wild, but there seems to be a preference for weekday releases.

@scor
Copy link

scor commented Jun 4, 2013

@greggles The changes in this PR do mention the 2 week period ("When the issue is not known to be exploited in the wild, a period of two weeks seems like a reasonable amount of time.") as well the preference for Drupal to release on Wednesdays. I guess the next question for @fabpot is whether it could be stated that Symfony would try to target Tue for security releases (for the reasons mentioned by greggles).

@scor
Copy link

scor commented Jun 4, 2013

On a related note, I think it would also help to start using this new collaboration model (once finalized) going forward before Drupal 8.0 is released, so we can validate it and get used to it. Drupal 8.0 is not going be released before the end of the year or beginning of 2014, so we have at least 6 months or so in front of us to start using this new model.

@greggles
Copy link

greggles commented Jun 4, 2013

Right you are. I hadn't noticed that this issue was associated with a commit. That said, the language still feels overly non-committal.

weaverryan added a commit that referenced this pull request Jun 22, 2013
added some more information about the security process
@weaverryan weaverryan merged commit cfa7e4c into symfony:2.1 Jun 22, 2013
@weaverryan
Copy link
Member

Hi guys!

I've merged this in - since it's for the documentation, it's more of a description of the currently-accepted model rather than what it perhaps should be, which I'm sure will continue :).

Thanks!

@fabpot fabpot deleted the security-tweaks branch March 3, 2014 12:24
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.

4 participants