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.

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

Add native tls support for actix_tls::connect module #295

Merged
merged 6 commits into from
Mar 27, 2021
Merged

Add native tls support for actix_tls::connect module #295

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
7f8b452
add native tisl support for actix_tls::connect module
fakeshadow Mar 24, 2021
dfe46a3
add changelog
fakeshadow Mar 24, 2021
9dde944
Update url.rs
robjtede Mar 24, 2021
bb83230
rename actix_tls::accept::nativetls module to native_tls
fakeshadow Mar 26, 2021
39b7570
update server changelog
robjtede Mar 26, 2021
0d146d0
fix stream lifetime
robjtede Mar 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update url.rs
  • Loading branch information
@robjtede
robjtede committed Mar 26, 2021
commit 9dde944b46955355a29d00ca12d0cc6eca60ac5a
2 changes: 1 addition & 1 deletion actix-router/src/url.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ impl Quoter {
}

cloned.map(|data| {
// Unsafe: we get data from http::Uri, which does utf-8 checks already
// SAFETY: we get data from http::Uri, which does UTF-8 checks already
// this code only decodes valid pct encoded values
unsafe { String::from_utf8_unchecked(data) }
})
Expand Down