Skip to content

Commit 5ce3f14

Browse files
committed
Auto merge of #3956 - Turbo87:headers, r=hi-rustin
Use `HeaderValue::from_static()` instead of `::from(&str)` This should be slightly more efficient 😄
2 parents de4f0a9 + 3653d5c commit 5ce3f14

File tree

2 files changed

+13
-4
lines changed

2 files changed

+13
-4
lines changed

src/admin/render_readmes.rs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,10 @@ pub fn run(opts: Opts) {
122122
let content = std::io::Cursor::new(readme);
123123
let readme_path = format!("readmes/{0}/{0}-{1}.html", krate_name, version.num);
124124
let mut extra_headers = header::HeaderMap::new();
125-
extra_headers.insert(header::CACHE_CONTROL, CACHE_CONTROL_README.parse().unwrap());
125+
extra_headers.insert(
126+
header::CACHE_CONTROL,
127+
header::HeaderValue::from_static(CACHE_CONTROL_README),
128+
);
126129
base_config
127130
.uploader()
128131
.upload(
@@ -165,7 +168,10 @@ fn get_readme(
165168
};
166169

167170
let mut extra_headers = header::HeaderMap::new();
168-
extra_headers.insert(header::USER_AGENT, USER_AGENT.parse().unwrap());
171+
extra_headers.insert(
172+
header::USER_AGENT,
173+
header::HeaderValue::from_static(USER_AGENT),
174+
);
169175
let response = match client.get(&location).headers(extra_headers).send() {
170176
Ok(r) => r,
171177
Err(err) => {

src/uploaders.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ impl Uploader {
145145
let mut extra_headers = header::HeaderMap::new();
146146
extra_headers.insert(
147147
header::CACHE_CONTROL,
148-
CACHE_CONTROL_IMMUTABLE.parse().unwrap(),
148+
header::HeaderValue::from_static(CACHE_CONTROL_IMMUTABLE),
149149
);
150150
self.upload(
151151
app.http_client(),
@@ -170,7 +170,10 @@ impl Uploader {
170170
let content_length = readme.len() as u64;
171171
let content = Cursor::new(readme);
172172
let mut extra_headers = header::HeaderMap::new();
173-
extra_headers.insert(header::CACHE_CONTROL, CACHE_CONTROL_README.parse().unwrap());
173+
extra_headers.insert(
174+
header::CACHE_CONTROL,
175+
header::HeaderValue::from_static(CACHE_CONTROL_README),
176+
);
174177
self.upload(
175178
http_client,
176179
&path,

0 commit comments

Comments
 (0)