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

feat/presentation exchange #4

Open
wants to merge 121 commits into
base: main
Choose a base branch
from

Conversation

anwalker293
Copy link

Using locustMediatorPresentProof.py, adds presentation exchange flows and scenarios to agent.js and locustClient.py. Currently, locustMediatorPresentProof.py is written such that the presentation exchange will happen, sequentially, after receiving first a credential.

Open to making changes! Please let me know!

ianco and others added 30 commits November 30, 2021 14:53
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
Signed-off-by: Kim Ebert <[email protected]>
Signed-off-by: Ian Costanzo <[email protected]>
- Networking changes introduced in Docker 4.1.x and forward on Windows and MAC stop the direct use of the internal docker host IP returned by the `docker run --rm --net=host eclipse/che-ip` process. On Windows and MAC `host.docker.internal` needs to be used for internal connections between containers on separate docker networks.
- `host.docker.internal` has been available on Windows and Mac since Docker Engine version 18.03 (March 2018). Support for `host.docker.internal` on Linux was introduced in version 20.10.0 (2020-12-08), but it does not run out of the box yet (as of Docker Engine 20.10.11 (2021-11-17)). You need to add `--add-host=host.docker.internal:host-gateway` to the `docker run` command in order for it to work.

Signed-off-by: Wade Barnes <[email protected]>
Add support for changes to internal Docker networking.
Signed-off-by: Kim Ebert <[email protected]>
- This is an update to the previous commit to update the support for Docker networking.  The `getDockerHost` has been pulled out and placed in a central location where it can be updated as needed rather than having to update dozens of scripts the next time there is a change.

Signed-off-by: Wade Barnes <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
jleach and others added 10 commits April 4, 2023 16:00
Signed-off-by: Jason C. Leach <[email protected]>
chore: update docs on how to supply a URL
* chore: update docs

Signed-off-by: Jason C. Leach <[email protected]>

* chore: update docs

Signed-off-by: Jason C. Leach <[email protected]>

* chore: updates docs for clarity

Signed-off-by: Jason C. Leach <[email protected]>

* chore: updates docs

Signed-off-by: Jason C. Leach <[email protected]>

* chore: updates docs

Signed-off-by: Jason C. Leach <[email protected]>

* chore: updates docs

Signed-off-by: Jason C. Leach <[email protected]>

* chore: updates docs

Signed-off-by: Jason C. Leach <[email protected]>

* chore: updates docs

Signed-off-by: Jason C. Leach <[email protected]>

---------

Signed-off-by: Jason C. Leach <[email protected]>
* Update ACA-Py set up to use Askar and Postgres 15

Signed-off-by: Stephen Curran <[email protected]>

* Update docs and settings in load testing sample.env

Signed-off-by: Stephen Curran <[email protected]>

---------

Signed-off-by: Stephen Curran <[email protected]>
Signed-off-by: Alex Walker <[email protected]>
@anwalker293
Copy link
Author

Important note: Verified state only checked for 5 seconds. We'll probably want to increase this

pres_ex_id = r['presentation_exchange_id']
# Want to do a for loop
iteration = 0
while iteration < 5:

Choose a reason for hiding this comment

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

Let us not use 5, but something like READ_TIMEOUT_SECONDS here?

@anwalker293
Copy link
Author

Note on changes made:

  • Included the variable VERIFIED_TIMEOUT_SECONDS (as opposed to READ_TIMEOUT_SECONDS, as this is already used). This is purely for checking the verified: true state. (If you want to make both the same variable, I can.) Default for the first variable is 20 seconds (can be changed upon request).
  • Propagated changes we made when taking care of crashing agents.
  • Made a new scenario, to match existing convention, on doing present proof with existing connections.
  • Cleaned up error handling
  • Put all sample commands within the docker-compose.yml. (This might need some review/recommendation on a better way to do this. As it stands, this won't work out of the box. (e.g. will have to delete the other lines))

I also verified that this works / nothing breaks by testing both of the changed scenarios. I will additionally make another, separate PR to put in the mediator message (for the 1kb test) now.

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