Skip to content

Commit fcda579

Browse files
committed
in_forward: Address review comments
Signed-off-by: Hiroshi Hatake <[email protected]>
1 parent 48d52cc commit fcda579

File tree

2 files changed

+4
-16
lines changed

2 files changed

+4
-16
lines changed

plugins/in_forward/fw.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -415,7 +415,7 @@ static struct flb_config_map config_map[] = {
415415
},
416416
{
417417
FLB_CONFIG_MAP_STR, "shared_key", NULL,
418-
0, FLB_FALSE, 0,
418+
0, FLB_TRUE, offsetof(struct flb_in_fw_config, shared_key),
419419
"Shared key for authentication"
420420
},
421421
{

plugins/in_forward/fw_config.c

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -86,24 +86,12 @@ struct flb_in_fw_config *fw_config_init(struct flb_input_instance *i_ins)
8686
}
8787

8888
/* Shared Key */
89-
p = flb_input_get_property("empty_shared_key", i_ins);
90-
if (p && flb_utils_bool(p)) {
91-
config->empty_shared_key = FLB_TRUE;
92-
}
93-
else {
94-
config->empty_shared_key = FLB_FALSE;
95-
}
96-
97-
p = flb_input_get_property("shared_key", i_ins);
9889
if (config->empty_shared_key) {
90+
if (config->shared_key) {
91+
flb_sds_destroy(config->shared_key);
92+
}
9993
config->shared_key = flb_sds_create("");
10094
}
101-
else if (p) {
102-
config->shared_key = flb_sds_create(p);
103-
}
104-
else {
105-
config->shared_key = NULL;
106-
}
10795

10896
/* Self Hostname */
10997
p = flb_input_get_property("self_hostname", i_ins);

0 commit comments

Comments
 (0)