Account created on 6 March 2008, about 17 years ago
#

Merge Requests

More

Recent comments

🇩🇪Germany webflo

Try to switch to "Alternate or no user endpoint" or "Microsoft Graph API (v1.0)" (make sure the User.Read scope is configured. The following adds the scope automatically on request.

🇩🇪Germany webflo

I think the cause of the problem is that the API call \Drupal\openid_connect_windows_aad\Plugin\OpenIDConnectClient\WindowsAad::buildUserinfo fails and this causes the profile data from ID and Access token.

🇩🇪Germany webflo

@mbopp and @ian_swan Could you provide a screenshot from the Drupal configuration? Thanks!

🇩🇪Germany webflo

🐛 No e-mail address provided by windows_aad after SSO authorization Needs review is a similar issue. I looks like some configurations are not supported anymore. Could someone post screenshots from the exact Azure configuration? Configured grants, access token/id token claims etc. This would help to reproduce the issue. Thanks!

🇩🇪Germany webflo

I wrote a test for it in branch "3515354-test-only"

🇩🇪Germany webflo

webflo made their first commit to this issue’s fork.

🇩🇪Germany webflo

I think the image download controller has to similar things. \Drupal\image\Controller\ImageStyleDownloadController::deliver uses \Drupal\image\Controller\ImageStyleDownloadController::getUriWithoutConvertedExtension

🇩🇪Germany webflo

webflo made their first commit to this issue’s fork.

🇩🇪Germany webflo

This is a small patch to address the double escaping from comment #2. Without getting rid of the DrupalTranslator class.

🇩🇪Germany webflo

webflo changed the visibility of the branch 3479031-update-js-component to active.

🇩🇪Germany webflo

The merge request is setup from the wrong branch. Don't know whats broken. Here is the patch.

🇩🇪Germany webflo

It is not necessary to add this filter specifically for media entities. EntityViewsData should check the owner key.

🇩🇪Germany webflo

Can we get a new release for the project? It has caused me a lot of problems when using the module. Unfortunately, everything was configured correctly, only the test mode was broken.

Great module! Thanks @ptmkenny

🇩🇪Germany webflo

I think it should be possible with custom code, but the access check is on the wrong form element.

🇩🇪Germany webflo

@catch Commit 43732e22 never made it Drupal 11. It is in 10.4.x and 10.5.x only

🇩🇪Germany webflo

Can you explain why the plugin configuration cannot be loaded? I think the problem is in your installation.

🇩🇪Germany webflo

I got bitten by this issue today. Roles for over 100 users have been deleted. I think needs an update hook, or should have an additional empty array condition during runtime.

Update in 🐛 Update role_mappings to avoid data loss Active

🇩🇪Germany webflo

@heikkiy I updated the config schema. Most of the properties you had in your list does not exist anymore. enable_single_sign_out got converted in openid_connect_windows_aad_update_9205 but we did not remove it from the config, but that is another issue.

🇩🇪Germany webflo

@kimberleycgm The patch does not work in every situation. The non B2C Endpoints end with "/oauth2/v2.0/authorize" as well.

🇩🇪Germany webflo

After consultation with the other maintainers, we have decided to only support Drupal 10.2+ and PHP 8.2+.

@pianomansam I am sorry, but I want to focus on stabilizing and modernizing. That it hard to to with Drupal 9 and PHP 7 in mind. I don't have the capacity to maintain two versions of the module.

🇩🇪Germany webflo

After consultation with the other maintainers, we have decided to only support Drupal 10.2+ and PHP 8.2+.

Thanks!

🇩🇪Germany webflo

webflo made their first commit to this issue’s fork.

🇩🇪Germany webflo

@sutharsan Thanks for the quick response! It looks like something went wrong. I don't have any additional permissions on drupal.org or gitlab to administer the project. Can you look into it?

Thanks!

🇩🇪Germany webflo

No, the module does not support v2 yet.

https://developer.friendlycaptcha.com/docs/v2/guides/upgrading-from-v1/ contains some docs how to do it. The difficulty will be to support both versions in one module.

🇩🇪Germany webflo

@liam morland Thanks! I messed the release process of version 1.9. Fixed it in 1.10

🇩🇪Germany webflo

webflo made their first commit to this issue’s fork.

🇩🇪Germany webflo

@amaria Thanks, sorry we missed it in the other PR.

🇩🇪Germany webflo

@khaled.zaidan Sorry, I overlooked your patch, I ended up with the same solution. Thank you!

🇩🇪Germany webflo

webflo changed the visibility of the branch 3412035-fix-project-update-status to active.

🇩🇪Germany webflo

webflo changed the visibility of the branch 3412035-fix-project-update-status to hidden.

🇩🇪Germany webflo

webflo made their first commit to this issue’s fork.

🇩🇪Germany webflo

Thanks to all. I have published a new release.

🇩🇪Germany webflo

webflo made their first commit to this issue’s fork.

🇩🇪Germany webflo

I think domain_source should not extend the DomainCommands class of domain, this will register the commands multiple times. But this was already the case before, we can solve it in another issue.

🇩🇪Germany webflo

I have merged the rebased feature branch, since the automatic rebase in gitlab failed.

Thanks @idebr for working on it.

🇩🇪Germany webflo

Tanks, to all. I have decided to remove DomainRouteProvider entirely (without BC layer). I think most since don't use it as extension point. It was never intended to be used as extension point. Those use cases are covered by CacheableRouteProviderInterface::addExtraCacheKeyPart.

I am sorry if this breaks a site.

🇩🇪Germany webflo

I have fixed DomainFormsTest.

🇩🇪Germany webflo

webflo changed the visibility of the branch 3477652-error-while-deleting to active.

🇩🇪Germany webflo

webflo changed the visibility of the branch 3477652-error-while-deleting to hidden.

🇩🇪Germany webflo

Thanks @rbrandon and @mpotter

Production build 0.71.5 2024