webflo → made their first commit to this issue’s fork.
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.
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.
@mbopp and @ian_swan Could you provide a screenshot from the Drupal configuration? Thanks!
🐛 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!
I wrote a test for it in branch "3515354-test-only"
I think the image download controller has to similar things. \Drupal\image\Controller\ImageStyleDownloadController::deliver uses \Drupal\image\Controller\ImageStyleDownloadController::getUriWithoutConvertedExtension
Thanks, merged in 3.x
This is a small patch to address the double escaping from comment #2. Without getting rid of the DrupalTranslator class.
webflo → changed the visibility of the branch 3479031-update-js-component to active.
The merge request is setup from the wrong branch. Don't know whats broken. Here is the patch.
It is not necessary to add this filter specifically for media entities. EntityViewsData should check the owner key.
webflo → created an issue. See original summary → .
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
I think it should be possible with custom code, but the access check is on the wrong form element.
@catch Commit 43732e22 never made it Drupal 11. It is in 10.4.x and 10.5.x only
Can you explain why the plugin configuration cannot be loaded? I think the problem is in your installation.
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
webflo → created an issue.
webflo → created an issue.
@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.
webflo → changed the visibility of the branch 3400940-fix-warning to active.
webflo → changed the visibility of the branch 3400940-fix-warning to hidden.
@kimberleycgm The patch does not work in every situation. The non B2C Endpoints end with "/oauth2/v2.0/authorize" as well.
Thanks, I reviewed it again. Looks good!
New MR against 3.1.x - https://git.drupalcode.org/project/menu_item_extras/-/merge_requests/38
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.
After consultation with the other maintainers, we have decided to only support Drupal 10.2+ and PHP 8.2+.
Thanks!
@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!
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.
@liam morland Thanks! I messed the release process of version 1.9. Fixed it in 1.10
Thanks to all.
@amaria Thanks, sorry we missed it in the other PR.
@khaled.zaidan Sorry, I overlooked your patch, I ended up with the same solution. Thank you!
New attempt, the explanation is in the code docs. Please review https://git.drupalcode.org/project/composer_deploy/-/merge_requests/9
webflo → changed the visibility of the branch 3412035-fix-project-update-status to active.
webflo → changed the visibility of the branch 3412035-fix-project-update-status to hidden.
Thanks to all. I have published a new release.
Fixed in dev already.
Thanks!
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.
I have merged the rebased feature branch, since the automatic rebase in gitlab failed.
Thanks @idebr for working on it.
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.
I have fixed DomainFormsTest.
webflo → changed the visibility of the branch 3477652-error-while-deleting to active.
webflo → changed the visibility of the branch 3477652-error-while-deleting to hidden.
Thanks @rbrandon and @mpotter