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

Allow symfony-7 in composer.json #188

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

mkilmanas
Copy link
Contributor

Changes

This is simply allowing Symfony 7 in composer.json - could not find any problems or incompatibilities at this point.

  • composer.json updated
  • all tests (composer test) are passing
  • tried it out on a minimal Symfony 7 project - no problems found
    • though I have only verified the JWT authentication use-case

References

#182

Testing

Attempted to install this bundle on a small Symfony project and perform JWT authentication. Other use-cases should be verified.

Maybe instead of merging to main branch, this should be "quarantined" in a dedicated branch (e.g. symfony7) of this main repository - that way other people would be able to install this version via composer by requiring auth0/symfony: dev-symfony7 and then test on their applications and their use-cases?

[ ] This change adds test coverage

[x] This change has been tested on the latest version of Symfony

Checklist

[x] I have read the Auth0 general contribution guidelines

[x] I have read the Auth0 Code of Conduct

[x] All existing and new tests complete without errors

@evansims
Copy link
Member

Hey @mkilmanas thanks a lot for contributing this; our team is at an offsite this week so my bandwidth is a little limited, but I'll take a look at this as soon as I'm back at my desk next week. Appreciate it!

@mkilmanas
Copy link
Contributor Author

@evansims any chance to review this anytime soon? auth0/symfony is now the only remaining dependency blocking us from migrating to Symfony 7 on our projects

@mkilmanas
Copy link
Contributor Author

Hey @evansims , is there anything happening at all? It's been over a month and no update.
Do we need to start considering a fork of this repo to have things moving? Or how else can we get support for up-to-date versions of Symfony?

@gusdeboer
Copy link

Will be needing this soon if our company was to implement Auth0

@ip512
Copy link

ip512 commented Feb 28, 2024

@evansims do you think this PR could be merged shortly ? This could be a showstopper in the adaption of auth0 for some projects if this SDK is not compatible with sf7

@mkilmanas
Copy link
Contributor Author

We have run out of patience and made a fork of this bundle -> https://packagist.org/packages/elvi/auth0-symfony

We will aim to maintain compatibility/updates as soon as they come to this official bundle, but also we will try to merge fixes/updates more proactively. No promises/guarantees though - still it's an open-source thing.

@gusdeboer @ip512 - feel free to give it a try (and any feedback welcome)

@evansims evansims merged commit d4c3aac into auth0:main Jun 24, 2024
18 checks passed
evansims added a commit that referenced this pull request Jun 24, 2024
This release includes experimental community-contributed support for Symfony 7. If you encounter any issues, please [open an issue on GitHub](https://github.com/auth0/symfony/issues).

**Added**

- Support for Symfony 7. [\#188](#188) ([mkilmanas](https://github.com/mkilmanas))
- Support string concatenation of scopes. [\#183](#183) ([mkilmanas](https://github.com/mkilmanas))

**Changed**

- Dashes in JWT permissions/scopes are now normalized. [\#184](#184) ([mkilmanas](https://github.com/mkilmanas))

**Fixed**

- Fixed an issue in controller constructors using a `$container` argument. [\#190)](#190) ([mkilmanas](https://github.com/mkilmanas))
@evansims evansims mentioned this pull request Jun 24, 2024
evansims added a commit that referenced this pull request Jun 24, 2024
This release includes experimental community-contributed support for
Symfony 7. If you encounter any issues, please [open an issue on
GitHub](https://github.com/auth0/symfony/issues).

**Added**

- Support for Symfony 7.
[\#188](#188)
([mkilmanas](https://github.com/mkilmanas))
- Support string concatenation of scopes.
[\#183](#183)
([mkilmanas](https://github.com/mkilmanas))

**Changed**

- Dashes in JWT permissions/scopes are now normalized.
[\#184](#184)
([mkilmanas](https://github.com/mkilmanas))

**Fixed**

- Fixed an issue in controller constructors using a `$container`
argument. [\#190)](#190)
([mkilmanas](https://github.com/mkilmanas))
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.

5 participants