Skip to content

MySQL create sessions table documentation incorrect #13764

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

Closed
wants to merge 1,725 commits into from
Closed

Conversation

beejaz
Copy link
Contributor

@beejaz beejaz commented Jun 7, 2020

This page hasnt been updated with this fix #12641 for "Preparing the Database to Store Sessions" on MySQL db, causing error:

SQLSTATE[22003]: Numeric value out of range: 1264 Out of range value for column 'sess_lifetime' at row 1

Its fixed by changing MEDIUMINT to INTEGER UNSIGNED for sess_lifetime column.

Its the same miss in 4.4, 5.0 and 5.1

javiereguiluz and others added 30 commits April 7, 2020 16:09
* 5.0:
  [String] Document the bytesAt() method
* 4.4:
  Update the metadata_update_threshold description
* 5.0:
  Update the metadata_update_threshold description
…ertStringContainsString (Wojciech Kania)

This PR was merged into the 5.0 branch.

Discussion
----------

[Testing] Change deprecated assertContains to assertStringContainsString

Symfony 5 requires PHP 7.2. PHPUnit Bridge for this version by default use [PHPUnit 8.3](https://symfony.com/doc/current/components/phpunit_bridge.html#modified-phpunit-script).
The assertContains is [deprecated](sebastianbergmann/phpunit#3425) and in PHPUnit 9 throws an error.

Commits
-------

e4d8c27 [Testing] Change deprecated assertContains to assertStringContainsString
* 5.0:
  [Testing] Change deprecated assertContains to assertStringContainsString
* 4.4:
  Avoid Supervisord to go in fatal error
* 5.0:
  Avoid Supervisord to go in fatal error
* 4.4:
  [Testing] Fixed the description of assertSelectorTextContains()
* 5.0:
  [Testing] Fixed the description of assertSelectorTextContains()
* 4.4:
  Update definitions.rst
  [EventDispatcher] Fix position of `AddEventAliasesPass`
  [Contributing] Fix `trigger_error()` position in deprecated class
* 5.0:
  Update definitions.rst
  [EventDispatcher] Fix position of `AddEventAliasesPass`
  [Contributing] Fix `trigger_error()` position in deprecated class
* 4.4:
  Mention that DoctrineCacheBundle is deprecated
* 5.0:
  Updated the Doctrine cache examples
  Mention that DoctrineCacheBundle is deprecated
…Store (kralos)

This PR was merged into the master branch.

Discussion
----------

#27345 Added Symfony\Component\Lock\Store\MongoDbStore

Added Lock MongoDbStore documentation to symfony 5.1

Commits
-------

934401f #27345 Added MongoDB Lock documentation
* 4.4:
  [symfony#13171] Some small rewordings
  [Security] Explain lazy anonymous mode
* 5.0:
  [symfony#13171] Some small rewordings
  [Security] Explain lazy anonymous mode
This PR was merged into the master branch.

Discussion
----------

Update Slack DSN

closes symfony#13531

Commits
-------

411b5c2 Update Slack DSN
* 4.4:
  [Validator] Fixed `Traverse` constraint reference
* 5.0:
  [Validator] Fixed `Traverse` constraint reference
* 4.4:
  Fixed yaml extension after conflicts
* 5.0:
  Fixed yaml extension after conflicts
This PR was merged into the master branch.

Discussion
----------

[Form] added the "choice_filter" option

Documentation for symfony/symfony#35733.

Based on symfony#13182 for now, so better be reviewed by commit here until it's merged and rebased, thanks!

Commits
-------

8b0c09e [Form] added the "choice_filter" option
* 4.4:
  Updated CODEOWNERS
* 5.0:
  Updated CODEOWNERS
javiereguiluz and others added 23 commits June 3, 2020 15:17
* 4.4:
  Update installation.rst
* 5.0:
  Update installation.rst
…ates automatically (javiereguiluz)

This PR was merged into the 5.1 branch.

Discussion
----------

[DI] Mention that container.no_preload tag propagates automatically

Fixes symfony#13583.

Commits
-------

062497a [DI] Mention that container.no_preload tag propagates automatically
…ax constraint (javiereguiluz)

This PR was submitted for the master branch but it was squashed and merged into the 5.1 branch instead.

Discussion
----------

[Validator] Documented the ExpressionLanguageSyntax constraint

@Andrej-in-ua could you please review this proposal to document the `ExpressionLanguageSyntax` constraint that you created? Thanks!

@xabbuh @weaverryan @wouterj while documenting this, I wondered if `names` and `validatedNames` options are overlapping a bit. Maybe we can merge them in a single option as follows?

```php
<?php
// Before

/**
 * @Assert\ExpressionLanguageSyntax(
 *     names = ['user', 'shipping_centers'],
 *     validateNames = true
 * )
 */
protected $shippingOptions;

// After

/**
 * @Assert\ExpressionLanguageSyntax(
 *     allowedVariables = ['user', 'shipping_centers']
 * )
 */
protected $shippingOptions;
```

Commits
-------

0984459 [Validator] Documented the ExpressionLanguageSyntax constraint
…s (weaverryan)

This PR was squashed before being merged into the 5.0 branch.

Discussion
----------

Adding details about Encore dev-server and https

Hi!

See: https://github.com/symfony/cli/issues/93

Just some missing docs on use the `dev-server` with the Symfony binary.

Commits
-------

828eb9b Adding details about Encore dev-server and https
* 5.0:
  Adding details about Encore dev-server and https
This PR was submitted for the 5.1 branch but it was merged into the 5.0 branch instead.

Discussion
----------

Update releases.rst

Bump up the version numbers

Commits
-------

68f1821 Bump versions of releases
* 5.0:
  Bump versions of releases
* 4.4:
  Update information in flex about added files for twig
* 5.0:
  Update information in flex about added files for twig
…ased Security system (wouterj)

This PR was squashed before being merged into the 5.1 branch.

Discussion
----------

[Security] Added docs about the authenticator-based Security system

As the new system is experimental, I think it's best to not update the main documentation of Security yet (also, the main guide isn't really affected by the new system - except from the removal of anonymous). This can be done once we deprecate the old system (in 5.3 or 5.4).

I do however think we should link more to this article, as I want to push people to use it. What do you think about adding a `.. tip::` in:

* [x] `security/guard_authentication` & `security/form_login_setup`
* [ ] <s>`security/custom_authentication_provider` (replace the one to Guard)</s>
* [x] `security` (at the start, as a "hey, please start by doing `enable_authenticator_manager: true`")

Fixes symfony#13575

Commits
-------

bdb5271 [Security] Added docs about the authenticator-based Security system
…capile)

This PR was squashed before being merged into the 5.1 branch.

Discussion
----------

[Security] anonymous: lazy deprecation warning

After upgrade to symfony/security-bundle to 5.1 and I got the deprecation warning Since symfony/security-bundle 5.1: Using "anonymous: lazy" to make the firewall lazy is deprecated, use "anonymous: true" and "lazy: true" instead.
symfony#13749

Commits
-------

73d1789 [Security] anonymous: lazy deprecation warning
* 4.4:
  Removed ClassLoader component docs
  Removed ErrorHandler component docs
  Removed Templating component docs
  Removed WebLink component docs
  Removed Polyfill component docs
  Removed Mercure component docs
  Removed DotEnv component docs
* 5.0:
  Removed ClassLoader component docs
  Removed ErrorHandler component docs
  Removed Templating component docs
  Removed WebLink component docs
  Removed Polyfill component docs
  Removed Mercure component docs
  Removed DotEnv component docs
* 5.0:
  Removed 4.4 versionadded directive
This page hasnt been updated with this fix symfony#12641 for "Preparing the Database to Store Sessions" causing error:
SQLSTATE[22003]: Numeric value out of range: 1264 Out of range value for column 'sess_lifetime' at row 1
@beejaz beejaz requested review from HeahDude and xabbuh as code owners June 7, 2020 10:08
@beejaz beejaz closed this Jun 7, 2020
@beejaz beejaz deleted the patch-1 branch June 7, 2020 10:12
@beejaz
Copy link
Contributor Author

beejaz commented Jun 7, 2020

Sorry, made the PR for 5.1 first and then changed it to 4.4 accidentally. Will create a new correct one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.