Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adx 998 - 3rd party update ckanext harvest to latest version #12

Draft
wants to merge 82 commits into
base: development
Choose a base branch
from

Conversation

A-Souhei
Copy link

@A-Souhei A-Souhei commented Jun 7, 2023

Description

Upgraded to ckanext-harvest from tag/v1.4.0 to tag/v1.5.4.
List of commands I used:

git clone git@github.com:fjelltopp/ckanext-harvest.git
cd ckanext-harvest
git remote add upstream https://github.com/ckan/ckanext-harvest.git
git fetch upstream
git checkout -b ADX-998-3rd-party-update-ckanext-harvest-to-latest-version
git merge tags/v1.5.4

Checklist

Put an x in the boxes that apply to this pull request (you can also fill these out after opening the pull request).
You may not need to check all boxes.

  • The Jira ticket for this issue has been updated to "Ready to Review" or equivalent.
  • I have developed these changes in discussion with the appropriate project manager.
  • My code follows the general Fjelltopp documentation (see Confluence).
  • I have made corresponding changes to the Fjelltopp documentation (see Confluence).
  • I have rebased this branch with master.
  • New dependency changes have been committed.
  • I have added automated tests that prove my fix is effective or that my feature works.
  • New and existing tests pass locally with my changes.
  • My changes generate no new warnings.
  • I have performed a self-review of my own code.
  • I have assigned at least one reviewer.

CKAN User and others added 30 commits April 22, 2022 18:22
Update harvester commands in README
…-list-page

Fix pagination for Dataset list on source page
Co-authored-by: Adrià Mercader <amercadero@gmail.com>
To match the vast majority of other extensions. pip-requiremetns.txt has
been kept as a symlink
add recipient only if there is email
To fix this failure on the CI tests:

```
   File "/__w/ckanext-harvest/ckanext-harvest/ckanext/harvest/harvesters/ckanharvester.py", line 7, in <module>
    from urllib3.contrib import pyopenssl
  File "/usr/lib/python3.9/site-packages/urllib3/contrib/pyopenssl.py", line 50, in <module>
    import OpenSSL.SSL
  File "/usr/lib/python3.9/site-packages/OpenSSL/__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL
  File "/usr/lib/python3.9/site-packages/OpenSSL/crypto.py", line 1550, in <module>
    class X509StoreFlags(object):
  File "/usr/lib/python3.9/site-packages/OpenSSL/crypto.py", line 1570, in X509StoreFlags
    CB_ISSUER_CHECK = _lib.X509_V_FLAG_CB_ISSUER_CHECK
AttributeError: module 'lib' has no attribute 'X509_V_FLAG_CB_ISSUER_CHECK'
```
…de-support-py2

Fixes unicode error in Python 2
add index harvest_error_harvest_object_id_idx
pdelboca and others added 29 commits March 3, 2023 10:48
This is a responsibility of the package. We are skiping any override since users will expect the behaviour of the custom logic added in package schema and validators.
Do not duplicate harvest_extras if exist in root schema
Use 403 when actions are forbidden, not 401
Fix url_for routing to point to harvester blueprint.
Fix asset path in MANIFEST.in
…uild

Fix a problem with data-dictization of lists to fix search-index rebuild
@A-Souhei A-Souhei marked this pull request as draft June 7, 2023 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants