Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

remove NOT NULL for client_secret #156

Open
goalain opened this issue Jan 17, 2017 · 1 comment
Open

remove NOT NULL for client_secret #156

goalain opened this issue Jan 17, 2017 · 1 comment

Comments

@goalain
Copy link

goalain commented Jan 17, 2017

In
oauth_clients -> client_secret character varying(80) NOT NULL,

And then in PDO:
public function isPublicClient($client_id) { $stmt = $this->db->prepare(sprintf('SELECT * from %s where client_id = :client_id', $this->config['client_table'])); $stmt->execute(compact('client_id')); if (!$result = $stmt->fetch(\PDO::FETCH_ASSOC)) { return false; } return empty($result['client_secret']); }

How can $result['client_secret'] be empty?
IMHO the "NOT NULL" is wrong as it is needed I.E. for 'allow_implicit' => true example.

@weierophinney
Copy link
Member

This repository has been closed and moved to laminas-api-tools/api-tools-oauth2; a new issue has been opened at laminas-api-tools/api-tools-oauth2#9.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants