ckanext-oauth2 icon indicating copy to clipboard operation
ckanext-oauth2 copied to clipboard

version upgrade for ckan2.9

Open liquanyong305 opened this issue 3 years ago • 7 comments

Hello everyone

We use Ckan2.9.4 to develop our project, but ckan-oauth2 only supports 2.7 or 2.8. Do you have any plans to upgrade version for ckan2.9?

Thanks a lot for the work you have done on this plugin.

liquanyong305 avatar Jan 18 '22 00:01 liquanyong305

Please test and give feedback to #46.

frafra avatar Feb 16 '22 09:02 frafra

Hi @frafra,

I tested #46 on latest CKAN v2.9.9 and v2.10.1 but I am getting below error: AttributeError AttributeError: 'Request' object has no attribute 'GET'

Any suggestion on this? Does any other PR exist, or are the latest changes in another branch?"

Nisha1293 avatar Aug 04 '23 11:08 Nisha1293

@nisha-nec See #42.

frafra avatar Aug 06 '23 09:08 frafra

Hi @frafra Thanks for the information. I tried but the code changes as per the CKANv2.9.x. I tested it on CKANv2.10.1, in this version, Authentication logic to use [Flask-login] instead of repoze.who. Do you have updated code. ? Any help will be appreciated.

Nisha1293 avatar Sep 01 '23 09:09 Nisha1293

@nisha-nec I use this one with CKAN 2.9: https://github.com/NINAnor/ckanext-oauth2/tree/aeae71acd20dffd49a3eb7b084a32523859b3212

frafra avatar Sep 04 '23 08:09 frafra

@frafra yes, in 2.9 it will work fine, but from CKAN 2.10, Repoze has been replaced for flask-login and this includes several changes:

  1. Login is now handled by user_login function (instead of set_repoze_user)
  2. auth_tkt cookie has been deprecated since it was being set by Repoze

Nisha1293 avatar Sep 04 '23 08:09 Nisha1293

@nisha-nec Sorry for the misunderstanding. As you can see, no one seems interested in supporting OAuth2 on newer CKAN. Someone needs to pick up the PR for 2.9, fix the remaining issues, then merge the PR (or fork the repository), and then focus on the 2.10.

frafra avatar Sep 04 '23 10:09 frafra