-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
ArgumentError: 'import/{{oauth2_generic_name}}' is not a supported controller name. #2101
Comments
same here with google. after upgrading from 12.7.7 to 12.7.8 gitlab will fail to start as long as the config contains google_oauth2 settings. #OAUTH OAUTH_ENABLED=true
|
@solidnerd before releasing 12.8 you should look at this issue there a missing else case and an extra line break in the template. I can’t send a pull request on my phone but it is easily to correct before new releases |
I did already a fix. I think I didn’t had pushed it 😅
Aurélien Chretien <[email protected]> schrieb am Sa. 28. März 2020
um 19:56:
… @solidnerd <https://github.com/solidnerd> before releasing 12.8 you
should look at this issue there a missing else case and an extra line break
in the template.
I can’t send a pull request on my phone but it is easily to correct before
new releases
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2101 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGYM32LUL7WYPRJVVP5MBLRJZB7BANCNFSM4LNVGZGQ>
.
|
…ed controller name Closes sameersbn#2101 Signed-off-by: solidnerd <[email protected]>
When an oAuth is configured (Azure in our case) the config file is wrongly generated.
There is no else case to remove the unused generic oauth part
docker-gitlab/assets/runtime/functions
Line 616 in 2ea27ee
The text was updated successfully, but these errors were encountered: