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

WIP: Initial commit to add TimeStamp sevice support #103

Closed
wants to merge 1 commit into from

Conversation

Strnadj
Copy link

@Strnadj Strnadj commented Oct 12, 2020

This MR relies on digitalbazaar/forge#824 because of unauthenticated attributes.

  • First implementation
  • Tests
  • Add authentication
  • Resolve replacing unauthenticated attributes in PKCS7 signature (better)

@stale
Copy link

stale bot commented Jan 10, 2021

This issue has been automatically marked as stale because it has not had activity in the past 90 days. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jan 10, 2021
@stale stale bot closed this Jan 20, 2021
@Hatzl
Copy link

Hatzl commented Mar 3, 2021

@Strnadj any reasons why this implementation is not done? I think it would be really helpful.

@vbuch
Copy link
Owner

vbuch commented Mar 4, 2021

I would guess that @Strnadj didn't complete it. It also relied on a still open PR in node-forge. The feature would require some effort in there as well.

@Hatzl
Copy link

Hatzl commented Mar 4, 2021

Thank you for your answer. I will try to tigger it. Maybe it helps..

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.

3 participants