Skip to content

[Doc] add explanations for encryption context in WebSocket #12810

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
Merged
Anonymitaet merged 4 commits into from
Nov 17, 2021
Merged

[Doc] add explanations for encryption context in WebSocket #12810

Anonymitaet merged 4 commits into from
Nov 17, 2021

Conversation

@Anonymitaet
Copy link
Member

@Anonymitaet Anonymitaet commented Nov 15, 2021

@Anonymitaet Anonymitaet added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Nov 15, 2021
@Anonymitaet Anonymitaet added this to the 2.10.0 milestone Nov 15, 2021
@Anonymitaet
Copy link
Member Author

Anonymitaet commented Nov 15, 2021 •
edited
Loading

@MarvinCai @Huanli-Meng can you review this PR? thanks

MarvinCai
MarvinCai reviewed Nov 15, 2021
View reviewed changes
Anonymitaet and others added 2 commits November 15, 2021 17:28
@Anonymitaet @MarvinCai
Update site2/docs/client-libraries-websocket.md
1a199ca 
Co-authored-by: Marvin Cai <zxc@streamnative.io>
@Anonymitaet
update 2
a98c3e0
MarvinCai
MarvinCai approved these changes Nov 15, 2021
View reviewed changes
Copy link
Contributor

@MarvinCai MarvinCai left a comment

Choose a reason for hiding this comment

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

lgtm

@Anonymitaet Anonymitaet merged commit 4c981af into apache:master Nov 17, 2021
@Anonymitaet Anonymitaet deleted the 1115 branch November 17, 2021 02:49
dlg99 pushed a commit to dlg99/pulsar that referenced this pull request Nov 23, 2021
@Anonymitaet @MarvinCai @dlg99
[Doc] add explanations for encryption context in WebSocket (apache#12810
e3c8e7e 
)

* [Doc] add explanations for encryption context in WebSocket

* update

* Update site2/docs/client-libraries-websocket.md

Co-authored-by: Marvin Cai <zxc@streamnative.io>

* update 2

Co-authored-by: Marvin Cai <zxc@streamnative.io>
eolivelli pushed a commit to eolivelli/pulsar that referenced this pull request Nov 29, 2021
@Anonymitaet @MarvinCai @eolivelli
[Doc] add explanations for encryption context in WebSocket (apache#12810
0a4874d 
)

* [Doc] add explanations for encryption context in WebSocket

* update

* Update site2/docs/client-libraries-websocket.md

Co-authored-by: Marvin Cai <zxc@streamnative.io>

* update 2

Co-authored-by: Marvin Cai <zxc@streamnative.io>
fxbing pushed a commit to fxbing/pulsar that referenced this pull request Dec 19, 2021
@Anonymitaet @MarvinCai
[Doc] add explanations for encryption context in WebSocket (apache#12810
9012fa4 
)

* [Doc] add explanations for encryption context in WebSocket

* update

* Update site2/docs/client-libraries-websocket.md

Co-authored-by: Marvin Cai <zxc@streamnative.io>

* update 2

Co-authored-by: Marvin Cai <zxc@streamnative.io>
@Anonymitaet Anonymitaet mentioned this pull request Dec 29, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@merlimat merlimat merlimat approved these changes

@MarvinCai MarvinCai MarvinCai approved these changes

@Huanli-Meng Huanli-Meng Awaiting requested review from Huanli-Meng

Assignees

@Anonymitaet Anonymitaet

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Milestone

2.10.0

Development

Successfully merging this pull request may close these issues.

3 participants

@Anonymitaet @merlimat @MarvinCai