Ldap Authorization: Implement only_ldap_authenticated option

Problem/Motivation

LDAP Authorization provides an option to bypass Authorization for non LDAP Users. The flag does not work atm,

Proposed resolution

Throw an exception for non LDAP users.

Remaining tasks

Decide on the implementation. Do it.

User interface changes

API changes

Data model changes

Comment	File	Size	Author
#11	2820852-11.patch	1.32 KB	grahl
#11	8.x-3.x: PHP 5.5 & MySQL 5.5, D8.3 2 pass PHP 5.5 & PostgreSQL 9.1, D8.3 2 pass PHP 5.5 & SQLite 3.8, D8.3 2 pass PHP 5.6 & MySQL 5.5, D8.3 2 pass PHP 7 & MySQL 5.5, D8.3 2 pass PHP 7 & PostgreSQL 9.1, D8.3 2 pass PHP 7 & SQLite 3.14, D8.3 2 pass
#2	2820852-2.patch	1.1 KB	webflo
#2	8.x-3.x: PHP 5.5 & MySQL 5.5, D8.3 Patch failed to apply PHP 5.5 & PostgreSQL 9.1, D8.3 Patch failed to apply PHP 5.5 & SQLite 3.8, D8.3 Patch failed to apply PHP 5.6 & MySQL 5.5, D8.3 Patch failed to apply PHP 7 & MySQL 5.5, D8.3 Patch failed to apply PHP 7 & PostgreSQL 9.1, D8.3 Patch failed to apply PHP 7 & SQLite 3.14, D8.3 Patch failed to apply

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

20 October 2016 at 15:50

webflo created an issue. See original summary.

Log in or register to post comments

Comment #2

webflo CreditAttribution: webflo at UEBERBIT GmbH commented 20 October 2016 at 15:50

Status:

Active

» Needs review

File	Size
2820852-2.patch	1.1 KB
8.x-3.x: PHP 5.5 & MySQL 5.5, D8.3 Patch failed to apply PHP 5.5 & PostgreSQL 9.1, D8.3 Patch failed to apply PHP 5.5 & SQLite 3.8, D8.3 Patch failed to apply PHP 5.6 & MySQL 5.5, D8.3 Patch failed to apply PHP 7 & MySQL 5.5, D8.3 Patch failed to apply PHP 7 & PostgreSQL 9.1, D8.3 Patch failed to apply PHP 7 & SQLite 3.14, D8.3 Patch failed to apply

Log in or register to post comments

Comment #3

webflo CreditAttribution: webflo at UEBERBIT GmbH commented 20 October 2016 at 15:50

Related issues:

+#2820847: Allow provides to skip the authorization certain scenarios

Log in or register to post comments

Comment #4

30 October 2016 at 20:37

Status:

Needs review

» Needs work

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

Comment #5

30 October 2016 at 20:37

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

Comment #6

30 October 2016 at 20:37

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

Comment #7

30 October 2016 at 20:37

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

Comment #8

30 October 2016 at 20:37

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

Comment #9

30 October 2016 at 20:37

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

Comment #10

30 October 2016 at 20:37

The last submitted patch, 2: 2820852-2.patch, failed testing.

Log in or register to post comments

grahl’s picture

Comment #11

he/him

Zürich

CreditAttribution: grahl as a volunteer commented 30 October 2016 at 21:03

Status:

Needs work

» Needs review

File	Size
2820852-11.patch	1.32 KB
8.x-3.x: PHP 5.5 & MySQL 5.5, D8.3 2 pass PHP 5.5 & PostgreSQL 9.1, D8.3 2 pass PHP 5.5 & SQLite 3.8, D8.3 2 pass PHP 5.6 & MySQL 5.5, D8.3 2 pass PHP 7 & MySQL 5.5, D8.3 2 pass PHP 7 & PostgreSQL 9.1, D8.3 2 pass PHP 7 & SQLite 3.14, D8.3 2 pass

1 file was hidden/shown/deleted

File	Size
2820852-2.patch	1.1 KB
8.x-3.x: PHP 5.5 & MySQL 5.5, D8.3 Patch failed to apply PHP 5.5 & PostgreSQL 9.1, D8.3 Patch failed to apply PHP 5.5 & SQLite 3.8, D8.3 Patch failed to apply PHP 5.6 & MySQL 5.5, D8.3 Patch failed to apply PHP 7 & MySQL 5.5, D8.3 Patch failed to apply PHP 7 & PostgreSQL 9.1, D8.3 Patch failed to apply PHP 7 & SQLite 3.14, D8.3 Patch failed to apply

Log in or register to post comments

grahl’s picture

Comment #12

he/him

Zürich

CreditAttribution: grahl as a volunteer commented 31 October 2016 at 14:21

Looks good. Could you commit the other patch to authorization today? I will follow suit right afterwards.

Log in or register to post comments

Comment #13

webflo CreditAttribution: webflo at UEBERBIT GmbH commented 31 October 2016 at 15:13

Alright, lets do it.

Log in or register to post comments

Comment #14

webflo CreditAttribution: webflo at UEBERBIT GmbH commented 31 October 2016 at 15:20

Committed #2820847: Allow provides to skip the authorization certain scenarios to the Authorization module

Log in or register to post comments

Comment #15

31 October 2016 at 15:25

grahl committed 87a2d4d on 8.x-3.x authored by webflo

Issue #2820852 by webflo, grahl: Ldap Authorization: Implement...

Log in or register to post comments

grahl’s picture

Comment #16

he/him

Zürich

CreditAttribution: grahl as a volunteer commented 31 October 2016 at 15:25

Status:

Needs review

» Fixed

Committed, thanks!

Log in or register to post comments

Comment #17

14 November 2016 at 15:34

Status:

Fixed

» Closed (fixed)

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

Log in or register to post comments