Francesco Frassinelli
Francesco Frassinelli
> Actually, the development is gone a little bit further and I solve some issues. Great! :) > Unfortunately, I didn't have time to address the changes requested. Some of...
It works just fine for me. I guess the review should be updated :)
Hi @FedericOldani! I can help with this PR if needed :)
It looks much better to me :) There are a couple of changes related to SSL, to force HTTPS I suppose.
Do you also have the same problem with the previous version?
@simao-silva that is a missing dependency that you haven't installed. If you cannot test a previous version, then it is hard to determine if you are hitting a regression.
@simao-silva try https://github.com/frafra/ckanext-oauth2/tree/flask_conversion_and_ckan_auth revision 01da0474c4f3f07edd5fba1a324168864ba4d86c from branch. It contains a previous snapshot of this PR.
@aitormagan any feedback on this PR?
@simao-silva I currently use revision 3633240df15112af591fbd7f87739681acf1af5a; you could try that one.
@aitormagan is unresponsive, and this is critical issue for the project, which seems dead to me. If nothing changes, we should just maintain a fork elsewhere. Any thoughts on that?