Commit Graph
184 Commits
Author SHA1 Message Date
Fabien Potencier 9ea56c3fcf added a new validation rule 2017-05-25 19:06:36 +02:00
Fabien Potencier 245d351019 bug #73 [lexik/jwt-authentication-bundle] Fix configured key path (chalasr)
This PR was merged into the master branch.

Discussion
----------

[lexik/jwt-authentication-bundle] Fix configured key path

Only using `%ETC_DIR%` leads to

> Private key "etc/jwt/private.pem" does not exist or is not readable.

This fixes it by prefixing paths by `%kernel.project_dir%/` in the bundle config.

Commits
-------

7f5180d [lexik/jwt-authentication-bundle] Fix configured key path
2017-05-24 08:11:00 +02:00
Fabien Potencier 41daf7dc64 bug #74 [doctrine/doctrine-bundle] Escape DATABASE_URL env value (chalasr)
This PR was merged into the master branch.

Discussion
----------

[doctrine/doctrine-bundle] Escape DATABASE_URL env value

When requiring `doctrine/doctrine-bundle` (more often `orm-pack`), the following is added to the project `.env`:

```
DATABASE_URL=mysql://root@127.0.0.1:3306/symfony?charset=utf8mb4&serverVersion=5.7
```
The makefile seems to parse that properly (tested on OSX and debian).
But running `source .env` from the cli (for debugging the .env vars, as shown in [dotenv docs](http://symfony.com/doc/master/components/dotenv.html)) gives

> .env:15: parse error near `&'

This fixes it by adding quotes around the value.

Commits
-------

1e60b1f [doctrine/doctrine-bundle] Escape DATABASE_URL env value
2017-05-24 08:09:39 +02:00
Robin Chalas 1e60b1f90d [doctrine/doctrine-bundle] Escape DATABASE_URL env value 2017-05-23 21:01:26 +02:00
Robin Chalas 7f5180d3f9 [lexik/jwt-authentication-bundle] Fix configured key path 2017-05-23 18:30:17 +02:00
Fabien Potencier 050655c481 feature #70 Remove version_aliases configuration setting (fabpot)
This PR was merged into the master branch.

Discussion
----------

Remove version_aliases configuration setting

`version_aliases` was a bad idea as it incurs too much manual maintenance as we need to remember to add new minor and major versions. That's fragile and we will probably forget to do it promptly. That's even worse for contributed recipes.

So, instead (and as already documented in the README, so not part of the diff), the version of the recipe if the minimum version that the recipe supports. So, if there is only one recipe for version "3.3", the recipe will be applied for all versions >= 3.3 (including 3.x, 4.x, ...).

If a version introduces changes that has an impact on the recipe , then a new directory with the version when the break appeared should be created with the new recipe, which makes total sense.

Commits
-------

07a5c67 removed version_aliases configuration setting
2017-05-20 09:55:30 +02:00
Fabien Potencier 07a5c67ace removed version_aliases configuration setting 2017-05-20 09:32:06 +02:00
Fabien Potencier 716ce4272d feature #65 Tweak configuration for app.yaml (fabpot)
This PR was merged into the master branch.

Discussion
----------

Tweak configuration for app.yaml

symfony/symfony-standard#1070

Commits
-------

7581f44 tweaked configuration for app.yaml
2017-05-20 07:50:02 +02:00
Fabien Potencier 11bd171055 bug #67 fix error exit code (xabbuh)
This PR was merged into the master branch.

Discussion
----------

fix error exit code

In case that the server could not start we should indicate the failure
with a non-zero exit code.

This addresses @stof's comment in https://github.com/symfony/recipes/pull/54#discussion_r116276643.

Commits
-------

9acb162 fix error exit code
2017-05-16 07:03:19 -07:00
Christian Flothmann 9acb162c7f fix error exit code
In case that the server could not start we should indicate the failure
with a non-zero exit code.
2017-05-16 09:23:11 +02:00
Fabien Potencier bed08a2c1a minor #66 Add a Github PR template (fabpot)
This PR was merged into the master branch.

Discussion
----------

Add a Github PR template

Commits
-------

a572667 added a PR template
2017-05-15 07:12:59 -07:00
Fabien Potencier a572667caa added a PR template 2017-05-14 10:00:44 -07:00
Fabien Potencier 7581f44af7 tweaked configuration for app.yaml 2017-05-14 09:33:30 -07:00
Fabien Potencier 2416e10255 synced phpunit config 2017-05-13 07:17:03 -07:00
Fabien Potencier 0967338d4d added a new validation rule 2017-05-12 14:19:57 -07:00
Fabien Potencier c8cc35795c feature #54 [FrameworkBundle] improve Makefile serve target (sstok)
This PR was merged into the master branch.

Discussion
----------

[FrameworkBundle] improve Makefile serve target

Related to #49, I don't know if this the best way to do it. But as the Makefile is composed I don't think it's possible to use advanced Makefile features 😟

@greg0ire suggested something like this:

```makefile
.PHONY: serve

CONSOLE=bin/console

$(CONSOLE):
    @echo 'Woops! The console does not exist'
    @exit

serve: bin/console
    bin/console list server|grep server:start
    bin/console server:start
```

But don't know if you can have multiple targets with the same name but different conditions?

Commits
-------

7a7de4b [FrameworkBundle] improve Makefile serve target
2017-05-12 08:13:04 -07:00
Sebastiaan Stok 7a7de4bd67 [FrameworkBundle] improve Makefile serve target 2017-05-12 17:03:31 +02:00
Fabien Potencier 0c159422a2 feature #62 Doctrine Migrations (Ricknox)
This PR was squashed before being merged into the master branch (closes #62).

Discussion
----------

Doctrine Migrations

This recipe adds doctrine migrations to the official Symfony recipes repository.

Commits
-------

52e1ebe Doctrine Migrations
2017-05-11 12:59:17 -07:00
Ricardo de VriesandFabien Potencier 52e1ebe6e8 Doctrine Migrations 2017-05-11 12:59:16 -07:00
Fabien Potencier 3d3eba7450 feature #60 [LexikJWTAuthenticationBundle] Add aliases (ogizanagi)
This PR was merged into the master branch.

Discussion
----------

[LexikJWTAuthenticationBundle] Add aliases

ping @chalasr

Commits
-------

4da5639 [LexikJWTAuthenticationBundle] Add aliases
2017-05-11 12:36:53 -07:00
Fabien Potencier 42c82ae6e3 bug #61 replaced php_sapi_name() with PHP_SAPI (fabpot)
This PR was merged into the master branch.

Discussion
----------

replaced php_sapi_name() with PHP_SAPI

Commits
-------

df6b8a0 replaced php_sapi_name() with PHP_SAPI
2017-05-10 16:16:29 -07:00
Fabien Potencier df6b8a04b8 replaced php_sapi_name() with PHP_SAPI 2017-05-10 11:19:42 -07:00
Fabien Potencier a9f9fc63fb fixed markup 2017-05-08 16:28:52 -07:00
Fabien Potencier 335a134166 fixed validation rule 2017-05-08 14:49:23 -07:00
Fabien Potencier eb6d9bbcd3 added new validation rules 2017-05-08 14:19:57 -07:00
Fabien Potencier 4484258e46 added a new validation rule 2017-05-08 10:47:08 -07:00
Fabien Potencier 80e81be8aa added a new validation rule 2017-05-08 10:22:09 -07:00
Fabien Potencier 29fe5f0045 added a new validation rule 2017-05-08 09:48:48 -07:00
Fabien Potencier 515e510a8b bug #58 Load container config, instead of importing it (ro0NL)
This PR was merged into the master branch.

Discussion
----------

Load container config, instead of importing it

`import()` is an implementation detail; see https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Config/Loader/GlobFileLoader.php#L26

Fixed ticket: https://github.com/symfony/symfony/pull/22638

Commits
-------

cecbb85 Load container config, instead of importing it
2017-05-08 07:01:58 -07:00
Fabien Potencier 40498fe62f bug #59 [PhpUnit] Use the new KERNEL_CLASS var for KernelTestCase (ogizanagi)
This PR was merged into the master branch.

Discussion
----------

[PhpUnit] Use the new KERNEL_CLASS var for KernelTestCase

Once https://github.com/symfony/symfony/pull/22668 is merged.

Commits
-------

c32f3e5 [PhpUnit] Use the new KERNEL_CLASS var for KernelTestCase
2017-05-08 07:00:38 -07:00
Maxime Steinhausser 4da56396f7 [LexikJWTAuthenticationBundle] Add aliases 2017-05-08 10:14:47 +02:00
Maxime Steinhausser c32f3e581a [PhpUnit] Use the new KERNEL_CLASS var for KernelTestCase 2017-05-08 10:11:53 +02:00
Roland FranssenandGitHub cecbb85b7c Load container config, instead of importing it 2017-05-07 12:23:08 +02:00
Fabien Potencier 549c14cd54 minor #51 Use single quotes in yaml files (yceruto)
This PR was merged into the master branch.

Discussion
----------

Use single quotes in yaml files

Commits
-------

b39aa63 Use single quotes in yaml files
2017-05-03 07:49:11 -07:00
Yonel Ceruto b39aa6303e Use single quotes in yaml files 2017-05-03 10:24:53 -04:00
Fabien Potencier 0301b52dcc minor #47 Remove framework.trusted_proxies option (ogizanagi)
This PR was merged into the master branch.

Discussion
----------

Remove framework.trusted_proxies option

As deprecated and "removed" in https://github.com/symfony/symfony/pull/21830 and https://github.com/symfony/symfony/pull/22238 (BC break)

Commits
-------

2507e41 Remove framework.trusted_proxies option
2017-05-03 07:09:27 -07:00
Fabien Potencier 19bb19b1a7 minor #45 Fixed useless calls (HeahDude)
This PR was merged into the master branch.

Discussion
----------

Fixed useless calls

Commits
-------

8e5d62d Fixed useless calls
2017-05-03 06:59:41 -07:00
Fabien Potencier 82d7b05797 bug #48 Enable interpretation of backslash escapes (sanpii)
This PR was merged into the master branch.

Discussion
----------

Enable interpretation of backslash escapes

Before:
![before](https://cloud.githubusercontent.com/assets/113045/25659624/ed7379ae-3008-11e7-9da4-c4272a3c49a8.png)

After:
![after](https://cloud.githubusercontent.com/assets/113045/25659629/f12a782c-3008-11e7-8128-9a55ea3204ad.png)

Commits
-------

d4832c2 Enable interpretation of backslash escapes
2017-05-03 06:58:40 -07:00
Sanpi d4832c2174 Enable interpretation of backslash escapes 2017-05-03 13:57:32 +02:00
Maxime SteinhausserandGitHub 2507e41e28 Remove framework.trusted_proxies option
As deprecated in https://github.com/symfony/symfony/pull/21830 and https://github.com/symfony/symfony/pull/22238 (BC break)
2017-05-03 13:00:26 +02:00
Jules Pietri 8e5d62d7b4 Fixed useless calls 2017-05-03 08:30:17 +02:00
Fabien Potencier f5bd6e00cd fixed admin recipe 2017-05-02 08:37:40 -07:00
Fabien Potencier 8996b40b92 minor #43 Add comments (GuilhemN)
This PR was merged into the master branch.

Discussion
----------

Add comments

Related to https://github.com/symfony/recipes/pull/42 and https://github.com/symfony/symfony-standard/pull/1064

Commits
-------

52c22bd Add comments
2017-05-01 16:33:01 -07:00
Fabien Potencier 5a50917e7d tweaked output of LexikJWTAuthenticationBundle 2017-05-01 16:30:36 -07:00
Fabien Potencier cf910ab116 feature #32 Add LexikJWTAuthenticationBundle (chalasr)
This PR was merged into the master branch.

Discussion
----------

Add LexikJWTAuthenticationBundle

Alternative to https://github.com/symfony/recipes-contrib/pull/5, I think it would be more appropriated here as this bundle has no competitor regarding the need it solves.
Repository: https://github.com/lexik/LexikJWTAuthenticationBundle

Commits
-------

9fdecab Add LexikJWTAuthenticationBundle for JWT Authentication in Symfony
2017-05-01 16:26:05 -07:00
Guilhem Niot 52c22bd05d Add comments 2017-05-01 19:29:21 +02:00
Fabien Potencier b6239744c5 bug #42 Store voters in Security (GuilhemN)
This PR was merged into the master branch.

Discussion
----------

Store voters in Security

[As caught by @xabbuh](https://github.com/symfony/symfony-standard/pull/1064#discussion_r114080504), in the docs, voters are stored in `Security`, not in `Voter`.

Commits
-------

6185425 Store voters in Security
2017-05-01 10:24:32 -07:00
Robin Chalas 9fdecab5ce Add LexikJWTAuthenticationBundle for JWT Authentication in Symfony 2017-05-01 12:03:29 +02:00
Guilhem NiotandGitHub 61854252b6 Store voters in Security 2017-04-30 20:38:31 +02:00
Fabien Potencier 5f2979d88d minor #37 Document the special %generate(secret)% env var value (javiereguiluz)
This PR was squashed before being merged into the master branch (closes #37).

Discussion
----------

Document the special %generate(secret)% env var value

Some recipes are starting to use this feature (see https://github.com/symfony/recipes-contrib/pull/5/files) so let's document it.

Commits
-------

8a103f4 Document the special %generate(secret)% env var value
2017-04-28 20:12:09 -07:00