search

DataShades / ckanext-saml2

SAML2 Authentication extension for CKAN
5 stars 23 forks source link

Define the official version of ckanext-saml2 #116

Open avdata99 opened 4 years ago

avdata99 commented 4 years ago

OKF also have a version

image

OKF created a 0.4 version wich also adds support for CKAN 2.8.

Task

amercader commented 4 years ago

"Official" is a loaded word that carries implications of support for the wider community, which is the reason why neither Link's or OKF's forks are in the main ckan GitHub org. To make an extension official at the very least it should have tests, which neither fork do AFAIA and to be honest a good internal refactoring wouldn't do harm. If Datopian has the capacity to tackle that it would be awesome to see.

Here are the changes we have added over time in our fork: https://github.com/DataShades/ckanext-saml2/compare/link-master...okfn:link-master. Some of it is opinionated but most stuff I think is valuable, so happy to help port it where it makes sense.

starsinmypockets commented 4 years ago

Maybe instead of saying "official" we can say "canonical" for now which would reduce duplication in effort?

starsinmypockets commented 4 years ago

The idea being to deprecate one of the repositories.