Skip to content

Commit 50b91c4

Browse files
author
Jake Champion
committed
make fields in DynamicBackendConfig be const
1 parent 0ebe314 commit 50b91c4

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

c-dependencies/js-compute-runtime/builtins/backend.cpp

+5-5
Original file line numberDiff line numberDiff line change
@@ -335,39 +335,39 @@ JS::Result<mozilla::Ok> Backend::register_dynamic_backend(JSContext *cx, JS::Han
335335
auto target_cstr = targetChars.get();
336336
if (hostOverride.has_value()) {
337337
backend_config_mask |= BACKEND_CONFIG_HOST_OVERRIDE;
338-
definition.host_override = (char *)hostOverride.value().c_str();
338+
definition.host_override = hostOverride.value().c_str();
339339
definition.host_override_len = hostOverride.value().length();
340340
} else {
341341
definition.host_override = nullptr;
342342
definition.host_override_len = 0;
343343
}
344344
if (certificateHostname.has_value()) {
345345
backend_config_mask |= BACKEND_CONFIG_CERT_HOSTNAME;
346-
definition.cert_hostname = (char *)certificateHostname.value().c_str();
346+
definition.cert_hostname = certificateHostname.value().c_str();
347347
definition.cert_hostname_len = certificateHostname.value().length();
348348
} else {
349349
definition.cert_hostname = nullptr;
350350
definition.cert_hostname_len = 0;
351351
}
352352
if (caCertificate.has_value()) {
353353
backend_config_mask |= BACKEND_CONFIG_CA_CERT;
354-
definition.ca_cert = (char *)caCertificate.value().c_str();
354+
definition.ca_cert = caCertificate.value().c_str();
355355
definition.ca_cert_len = caCertificate.value().length();
356356
} else {
357357
definition.ca_cert = nullptr;
358358
definition.ca_cert_len = 0;
359359
}
360360
if (ciphers.has_value()) {
361361
backend_config_mask |= BACKEND_CONFIG_CIPHERS;
362-
definition.ciphers = (char *)ciphers.value().c_str();
362+
definition.ciphers = ciphers.value().c_str();
363363
definition.ciphers_len = ciphers.value().length();
364364
} else {
365365
definition.ciphers = nullptr;
366366
definition.ciphers_len = 0;
367367
}
368368
if (sniHostname.has_value()) {
369369
backend_config_mask |= BACKEND_CONFIG_SNI_HOSTNAME;
370-
definition.sni_hostname = (char *)sniHostname.value().c_str();
370+
definition.sni_hostname = sniHostname.value().c_str();
371371
definition.sni_hostname_len = sniHostname.value().length();
372372
} else {
373373
definition.sni_hostname = nullptr;

c-dependencies/js-compute-runtime/xqd.h

+5-5
Original file line numberDiff line numberDiff line change
@@ -74,20 +74,20 @@ typedef enum TLS {
7474
} TLS;
7575

7676
typedef struct DynamicBackendConfig {
77-
char *host_override;
77+
const char *host_override;
7878
uint32_t host_override_len;
7979
uint32_t connect_timeout_ms;
8080
uint32_t first_byte_timeout_ms;
8181
uint32_t between_bytes_timeout_ms;
8282
uint32_t ssl_min_version;
8383
uint32_t ssl_max_version;
84-
char *cert_hostname;
84+
const char *cert_hostname;
8585
uint32_t cert_hostname_len;
86-
char *ca_cert;
86+
const char *ca_cert;
8787
uint32_t ca_cert_len;
88-
char *ciphers;
88+
const char *ciphers;
8989
uint32_t ciphers_len;
90-
char *sni_hostname;
90+
const char *sni_hostname;
9191
uint32_t sni_hostname_len;
9292
} DynamicBackendConfig;
9393

0 commit comments

Comments
 (0)