laminas-api-tools / api-tools-oauth2

Laminas module for implementing an OAuth2 server
https://api-tools.getlaminas.org/documentation
BSD 3-Clause "New" or "Revised" License
11 stars 15 forks source link

Remove IbmDb2 Adapter #21

Closed alexdenvir closed 4 years ago

alexdenvir commented 4 years ago
Q A
Documentation no
Bugfix no
BC Break no?
New Feature no
RFC no
QA yes

Description

Removes IbmDb2 Adapter. This is basically just a copy of https://github.com/zfcampus/zf-oauth2/pull/168 which was never merged, so credit should go to @michalbundyra. As per the original PR, the IbmDb2 storage is not in the oauth2 library, and never has been. The original PR to add this storage to the oauth2 library was opened in 2015, and last updated 2017.

I don't think this is a BC break, but it is removing a class regardless so maybe thats a judgement call for maintainers?

Fixes #3, and fixes broken tests with coverage (~with my recently merged PR, builds should start passing again!~ cs checks still fail :cry:)