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

cool#9992 doc electronic sign: fetch the created signature #10609

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

vmiklos
Copy link
Contributor

@vmiklos vmiklos commented Nov 27, 2024

Sign a hash with electronic signing, we get a postmessage that the
signing is done on the external site, but we still don't have the
signed hash.

There is a /api/signatures/download-signed-file endpoint to fetch the
PKCS#7 blob, but that is only meant to succeed once we know the signing
happened.

Handle this by fetching the signature once we got a postmessage that
the signing was fine.

The blob we get still needs integrating, though.

Signed-off-by: Miklos Vajna vmiklos@collabora.com
Change-Id: Id0dcdf5a6518c6b71ab02f1f77a3736038546821

@vmiklos
Copy link
Contributor Author

vmiklos commented Nov 27, 2024

https://cpci.cbg.collabora.co.uk:8080/job/github_online_master_debug_vs_co-24.04_cypress_desktop/2919/console fails in:

calc/searchbar_spec.js

but a local

make -C cypress_test -sr check-desktop spec=calc/searchbar_spec.js

passes, retry.

@vmiklos
Copy link
Contributor Author

vmiklos commented Nov 27, 2024

https://cpci.cbg.collabora.co.uk:8080/job/github_online_master_debug_vs_co-24.04_cypress_desktop/2922/console again fails in:

calc/searchbar_spec.js

but turns out my local baseline was newer when I ran cypress (vs the one I used for pushing), rebase.

@vmiklos
Copy link
Contributor Author

vmiklos commented Nov 27, 2024

https://cpci.cbg.collabora.co.uk:8080/job/github_online_master_debug_vs_co-24.04_cypress_desktop/2924/console still fails in:

calc/searchbar_spec.js

hmm, let me try yesterday's baseline to see if that helps.

Sign a hash with electronic signing, we get a postmessage that the
signing is done on the external site, but we still don't have the
signed hash.

There is a /api/signatures/download-signed-file endpoint to fetch the
PKCS#7 blob, but that is only meant to succeed once we know the signing
happened.

Handle this by fetching the signature once we got a postmessage that
the signing was fine.

The blob we get still needs integrating, though.

Signed-off-by: Miklos Vajna <vmiklos@collabora.com>
Change-Id: Id0dcdf5a6518c6b71ab02f1f77a3736038546821
@vmiklos
Copy link
Contributor Author

vmiklos commented Nov 27, 2024

@caolanm could you please review this? Thanks.

If the above cypress problem starts to be a pattern, we should probably investigate, but it's not something I see locally. Apart from that, only a last step is meant to be missing for the "have 1 working scenario" of this e-sign thing.

Copy link
Contributor

@caolanm caolanm left a comment

Choose a reason for hiding this comment

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

I have seen calc searchbar a few times, chris attempted a fix recently I think

@caolanm caolanm merged commit fb78418 into master Nov 27, 2024
13 checks passed
@caolanm caolanm deleted the private/vmiklos/master branch November 27, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants