Problem/Motivation

In 6.0.2, https://www.drupal.org/project/simple_oauth/issues/3547975 fixed routes not supporting the format html.

I am afraid that with 6.0.3 we have almost come full-circle on this.
Now routes that do not allow the default format "html", like typical REST routes, are broken again:

With https://www.drupal.org/project/simple_oauth/issues/3518947, the "applies()" method
returns FALSE in those cases which leads to a 403. Since no oauth2 authentication takes place,
I guess.

What's more: $url_object should be initialized to NULL before the "try".
I get "Warning: Undefined variable $url_object in Drupal\simple_oauth\Authentication\Provider\SimpleOauthAuthenticationProvider->applies()" (line 136 of /var/www/html/web/modules/contrib/simple_oauth/src/Authentication/Provider/SimpleOauthAuthenticationProvider.php)

Line 136 is the one with
if (!$url_object) {

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

cspitzlay created an issue. See original summary.

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

alorenc’s picture

Assigned: Unassigned » alorenc
alorenc’s picture

Assigned: alorenc » Unassigned
Status: Active » Needs work
claudiu.cristea’s picture

Priority: Normal » Critical

This is critical

bojan_dev’s picture

Status: Needs work » Fixed
Related issues: +#3518947: 404 error not output correctly.

My bad, I have reverted #3518947: 404 error not output correctly. and made a new release tag.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.