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.

Sign up for GitHub

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

Jump to bottom

Reuse indy-node-container image #112

Closed
wants to merge 75 commits into from
Closed

Reuse indy-node-container image #112

wants to merge 75 commits into from

Conversation

c2bo
Copy link
Member

@c2bo c2bo commented Nov 10, 2022

Implements #111

domwoe added 30 commits February 21, 2022 10:48
@domwoe
did:indy resolver
8b89763 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
new nym request
233e050 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
do not add resolver in legacy mode
3d8b5fe 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
sync with IDunion/indy-did-resolver
a4fddcd 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
renaming Value to SJsonValue
98b088d 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
invert feature flag logic: legacy -> did_indy
9d993de 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
format
a7c1dd5 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
small fix in test dependencies
37887ac 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
nym and get_nym + proxy. No wrappers
cfb948f 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
adapt python wrapper
f52d3a0 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
remove feature
f71d03b 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
minimal changes to nodejs wrapper
01df1d8 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
minimal changes to go wrapper
7e922c1 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
test build
84aafbd 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
format
8ed4a0f 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
fix nym builder in tests and try to fix go wrapper
d8427ed 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
another attempt to fix go wrapper
e9c8ce2 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
not working due to lifetimes
2c9d6c1 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
run tests
139c538 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
ffi resolver working for single request
041fd8d 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
optional parameters in get_nym
581c00b 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
started clean up of ffi resolver
71ca5d6 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
remove chrono dependency and use time-rs instead
c478b5f 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
simplifying module imports
af33d9c 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
format
8a73927 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
ffi without fetching legacy endpoint working
d6a5fb4 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
fix query params in proxy
18b774f 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
format..
20798a4 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
remove some unecessary code
f7fe8d4 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
removed unecessary enum
49bf453 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
domwoe and others added 16 commits March 29, 2022 17:12
@domwoe
small fixes
5d0fa6a 
Signed-off-by: Dominic Woerner <dom.woe@gmail.com>
@domwoe
Get attrib with optional seq_no and timestamp
f7585dd 
Signed-off-by: Dominic Woerner <dom.woe@gmail.com>
@domwoe
fix did with query parameters
390db9e 
Signed-off-by: Dominic Woerner <dom.woe@gmail.com>
@domwoe
update local_nodes_pool attrib tests
7093808 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
remove debug loglevel
7a82ad4 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
Merge branch 'feature/self-cert-version-tests' into feature/get-attri…
d3d684a 
…b-versions
@domwoe
Merge pull request hyperledger#4 from domwoe/feature/get-attrib-versions
92d4311 
Get attrib with optional seq_no and timestamp and self-cert version tests
@domwoe
attempt to harden resolver test
5e1e95a 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@domwoe
fmt
d1cf24e 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@andrewwhitehead
Merge pull request hyperledger#87 from domwoe/feature/python-with-mul…
f76735d 
…tiple-ledger-support

Feature/python wrapper with multiple ledger support
@domwoe
fix readme for indy-vdr-proxy
f538397 
Signed-off-by: Dominic Wörner <dom.woe@gmail.com>
@andrewwhitehead
Merge pull request hyperledger#90 from domwoe/fix/flag-instead-arg
1470f0d 
fix readme for indy-vdr-proxy
@c2bo
Correctly use the networks folder of the repository
5161a25 
Signed-off-by: Christian Bormann <ChristianCarl.Bormann@de.bosch.com>
@andrewwhitehead
Merge pull request hyperledger#105 from boschresearch/fix/repository-…
0544131 
…structure

Correctly use the networks folder of the repository
@c2bo
fix verkey expansion
f2483cf 
Signed-off-by: Christian Bormann <ChristianCarl.Bormann@de.bosch.com>
@andrewwhitehead
Merge pull request hyperledger#104 from boschresearch/fix/expand-verkey
f1345bb 
fix verkey expansion
andrewwhitehead
andrewwhitehead reviewed Nov 10, 2022
View reviewed changes
ci/indy-pool.dockerfile Outdated
software-properties-common

RUN pip3 install -U \
RUN pip3 install \
"pip~=9.0" \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think whatever pip and setuptools is in the image should be sufficient?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, seems to work with only supervisor

@c2bo
Re-use indy-node-container images as base for system tests
a8cc16e 
Signed-off-by: Christian Bormann <ChristianCarl.Bormann@de.bosch.com>
@c2bo c2bo force-pushed the feature/reuse-indy-node-image branch from 3d7b94f to a8cc16e Compare November 11, 2022 12:16
@c2bo
Copy link
Member Author

c2bo commented Nov 14, 2022

The images of the indy-node-container will be renamed whenever hyperledger/indy-node-container#121 is merged. I'd prefer waiting for this change and adjust this PR before merging.

@c2bo
Use new naming convention of indy-node-container
17af328 
Signed-off-by: Christian Bormann <ChristianCarl.Bormann@de.bosch.com>
@c2bo c2bo marked this pull request as ready for review January 17, 2023 16:04
@c2bo c2bo mentioned this pull request Feb 7, 2023
Closed
c2bo added 2 commits February 15, 2023 15:43
@c2bo
extend the bash script for local testing
52c9216 
Signed-off-by: Christian Bormann <ChristianCarl.Bormann@de.bosch.com>
@c2bo
use new rc5 version for tests
4345376 
Signed-off-by: Christian Bormann <ChristianCarl.Bormann@de.bosch.com>
@c2bo c2bo mentioned this pull request Mar 1, 2023
Merged
@c2bo c2bo changed the base branch from did-indy to did-indy-merge March 2, 2023 05:44
@c2bo c2bo closed this Mar 2, 2023
@c2bo c2bo mentioned this pull request Mar 2, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrewwhitehead andrewwhitehead andrewwhitehead approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@c2bo @andrewwhitehead @domwoe