-
-
Notifications
You must be signed in to change notification settings - Fork 632
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
ecphp/cas-bundle: add recipe for version 3.0 #1687
Conversation
drupol
commented
Oct 9, 2024
•
edited
Loading
edited
Q | A |
---|---|
License | MIT |
Packagist | https://packagist.org/packages/ecphp/cas-bundle |
Thanks for the PR 😍 How to test these changes in your application
Diff between recipe versionsIn order to help with the review stage, I'm in charge of computing the diff between the various versions of patched recipes. ecphp/cas-bundle1.0 vs 2.02.0 vs 3.0diff --git a/ecphp/cas-bundle/3.0/config/packages/dev/cas_bundle.yaml b/ecphp/cas-bundle/3.0/config/packages/dev/cas_bundle.yaml
new file mode 100644
index 00000000..5697bf90
--- /dev/null
+++ b/ecphp/cas-bundle/3.0/config/packages/dev/cas_bundle.yaml
@@ -0,0 +1,24 @@
+cas:
+ # Change the URL to your own CAS server
+ base_url: https://ecas.ec.europa.eu/cas
+ protocol:
+ login:
+ path: /login
+ default_parameters:
+ service: cas_bundle_homepage
+ serviceValidate:
+ path: /p3/serviceValidate
+ default_parameters:
+ format: JSON
+ #pgtUrl: cas_bundle_proxy_callback
+ logout:
+ path: /logout
+ default_parameters:
+ service: cas_bundle_homepage
+ proxy:
+ path: /proxy
+ proxyValidate:
+ path: /p3/proxyValidate
+ default_parameters:
+ format: JSON
+ #pgtUrl: cas_bundle_proxy_callback
diff --git a/ecphp/cas-bundle/3.0/config/packages/dev/cas_security.yaml b/ecphp/cas-bundle/3.0/config/packages/dev/cas_security.yaml
new file mode 100644
index 00000000..9595b42e
--- /dev/null
+++ b/ecphp/cas-bundle/3.0/config/packages/dev/cas_security.yaml
@@ -0,0 +1,4 @@
+security:
+ providers:
+ cas:
+ id: EcPhp\CasBundle\Security\Core\User\CasUserProvider
diff --git a/ecphp/cas-bundle/3.0/config/routes/cas.yaml b/ecphp/cas-bundle/3.0/config/routes/cas.yaml
new file mode 100644
index 00000000..a8ec5e45
--- /dev/null
+++ b/ecphp/cas-bundle/3.0/config/routes/cas.yaml
@@ -0,0 +1,3 @@
+cas_bundle:
+ resource: "@CasBundle/Resources/config/routes/routes.php"
+ prefix: /cas
diff --git a/ecphp/cas-bundle/2.0/manifest.json b/ecphp/cas-bundle/3.0/manifest.json
index 272a15d0..33e525ce 100644
--- a/ecphp/cas-bundle/2.0/manifest.json
+++ b/ecphp/cas-bundle/3.0/manifest.json
@@ -1,8 +1,10 @@
{
"bundles": {
- "EcPhp\\CasBundle\\CasBundle": ["all"]
+ "EcPhp\\CasBundle\\CasBundle": [
+ "all"
+ ]
},
- "copy-from-package": {
- "Resources/config/": "%CONFIG_DIR%/"
+ "copy-from-recipe": {
+ "config/": "%CONFIG_DIR%/"
}
} |
Head branch was pushed to by a user without write access
6493355
to
70b34fb
Compare
The build issue make sense since we let the user choosing with which Installing this bundle is usually done by doing: |
Cool, thanks! |