Skip to content

Comments

Experimental -- v4 provider#609

Merged
calina-c merged 73 commits intov4mainfrom
experimental-provider-validator
Nov 11, 2021
Merged

Experimental -- v4 provider#609
calina-c merged 73 commits intov4mainfrom
experimental-provider-validator

Conversation

@calina-c
Copy link
Contributor

@calina-c calina-c commented Nov 4, 2021

List of PR changes:

  • replace old ECIES_PRIVATE_KEY with PRIVATE_KEY. The purpose of the PRIVATE_KEY is not to decrypt, but to identify the Aquarius version when asking for decrypt from Provider
  • temporarily added the artifacts folder and artifacts/install.sh script until we have an ocean-contracts release
  • where functions depended on the old DDO structure, replace with the new one, e.g. instead of using get_metadata_from_services, we can now use the "metadata" key directly. Other mischellaneous changes related to the structure, as needed (not all listed below):
    • replace the date format with ISO Format
  • remove the encrypt_ddo and encrypt_as_hex endpoints and the encrypt_data function
  • repurpose the decryptor class to contain utilitaries for “talking to” Provider
  • remove metadata contract from events monitor, since there are now multiple contracts
  • reconfigure the event logs retrieval in events monitor to filter by topics for MetadataCreated and MetadataUpdated events, regardless of contract
  • replace some logic in processors.py to handle the new structure/event arguments
  • remove eciespy from dependencies (no longer needed)

Changes in tests:

  • remove deployment of datatoken contract
  • remove rawddos (pertaining to old processor tests)
  • repurpose send_create_update_tx function to use setMetaData
  • use v4 samples
  • add unit and functional tests where needed

TODOs:

  • check_no_urls_in_files -> adapt
  • remove curation :D
  • check all arguments for events processors are used
  • clear v3 samples ?
  • add "real" metadata start block: should be taken from barge, but it is not currently written in the address file

@calina-c calina-c requested a review from a team as a code owner November 4, 2021 12:50
@calina-c
Copy link
Contributor Author

calina-c commented Nov 4, 2021

Screenshot 2021-11-04 at 14 49 40

@calina-c calina-c changed the title Experimental provider validator Experimental -- v4 provider Nov 4, 2021
@calina-c calina-c force-pushed the experimental-provider-validator branch from 88ad216 to 43a8b35 Compare November 5, 2021 13:34
@calina-c calina-c force-pushed the experimental-provider-validator branch from 24bfa12 to 4205f2b Compare November 8, 2021 08:59
@calina-c calina-c force-pushed the experimental-provider-validator branch from 69f6c64 to f4da928 Compare November 10, 2021 08:08
@calina-c calina-c merged commit 108e00f into v4main Nov 11, 2021
@calina-c calina-c deleted the experimental-provider-validator branch November 11, 2021 07:26
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.

2 participants