Skip to content

EDU-1072: Clarify the key to be used to convert in PKCS8 for Java SDK#2301

Merged
flossypurse merged 2 commits intomainfrom
taonic/update_cert_pkcs8_instruction
Aug 25, 2023
Merged

EDU-1072: Clarify the key to be used to convert in PKCS8 for Java SDK#2301
flossypurse merged 2 commits intomainfrom
taonic/update_cert_pkcs8_instruction

Conversation

@taonic
Copy link
Copy Markdown
Contributor

@taonic taonic commented Aug 22, 2023

What does this PR do?

To make it clear that the key to be converted to PKCS8 is the entity's private key.

Some users confused this with the CA key: https://temporalio.slack.com/archives/CTT84KXK9/p1692291446276049

@taonic taonic requested a review from a team as a code owner August 22, 2023 02:40
@djmagee djmagee changed the title Clarify the key to be used to convert in PKCS8 for Java SDK EDU-1072: Clarify the key to be used to convert in PKCS8 for Java SDK Aug 22, 2023
@flossypurse flossypurse enabled auto-merge (squash) August 25, 2023 15:04
@flossypurse flossypurse merged commit e16ce66 into main Aug 25, 2023
@flossypurse flossypurse deleted the taonic/update_cert_pkcs8_instruction branch August 25, 2023 15:04
flippedcoder pushed a commit that referenced this pull request May 4, 2026
…elds (#4526)

* docs: relax host resource reporting requirement; refresh audit log fields

- worker-health: drop the resource-based tuner requirement for host CPU/memory
  reporting and trim the Go SysInfoProvider example to a minimal worker.Options
  setup, matching how other SDKs document this. SysInfoProvider can now be
  set directly on worker.Options (sdk-go #2301).
- audit-logs: remove the previously deprecated fields (user_email, level,
  caller_ip_address, details, category) from the format spec, deprecate
  request_id with a November 1 2026 removal target, and document
  async_operation_id as its replacement.

* docs: call out sysinfo contrib import in host resource reporting example

The trimmed example dropped the imports entirely, which obscured that
sysinfo is a contrib package and needs to be pulled in explicitly. Add the
relevant imports back to the snippet and a short prose note explaining why.
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