Skip to content

Commit 03cfc04

Browse files
committed
fix: pass->password
1 parent 7889e6b commit 03cfc04

File tree

2 files changed

+32
-26
lines changed

2 files changed

+32
-26
lines changed

plugins/restful/normal_handle.c

-23
Original file line numberDiff line numberDiff line change
@@ -218,18 +218,6 @@ void handle_update_user(nng_aio *aio)
218218
return;
219219
}
220220

221-
// old password length check
222-
int old_pass_len = strlen(req->old_pass);
223-
if (old_pass_len < NEU_USER_PASSWORD_MIN_LEN ||
224-
old_pass_len > NEU_USER_PASSWORD_MAX_LEN) {
225-
nlog_error("user `%s` old password too short or too long",
226-
req->name);
227-
NEU_JSON_RESPONSE_ERROR(NEU_ERR_INVALID_PASSWORD_LEN, {
228-
neu_http_response(aio, error_code.error, result_error);
229-
});
230-
return;
231-
}
232-
233221
// only admin & current user can update user
234222
if (0 != strcmp(req->name, "admin") &&
235223
0 != strcmp(req->name, current_user)) {
@@ -250,17 +238,6 @@ void handle_update_user(nng_aio *aio)
250238
return;
251239
}
252240

253-
// check old password
254-
if (!neu_user_check_password(user, req->old_pass)) {
255-
nlog_error("user `%s` old password check fail", req->name);
256-
NEU_JSON_RESPONSE_ERROR(NEU_ERR_INVALID_USER_OR_PASSWORD, {
257-
neu_http_response(aio, error_code.error, result_error);
258-
});
259-
260-
neu_user_free(user);
261-
return;
262-
}
263-
264241
// update user password
265242
int rv = neu_user_update_password(user, req->new_pass);
266243
if (0 != rv) {

src/parser/neu_json_login.c

+32-3
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ int neu_json_decode_add_user_req(char *buf, neu_json_add_user_req_t **result)
143143
.t = NEU_JSON_STR,
144144
},
145145
{
146-
.name = "pass",
146+
.name = "password",
147147
.t = NEU_JSON_STR,
148148
},
149149

@@ -175,12 +175,41 @@ void neu_json_decode_add_user_req_free(neu_json_add_user_req_t *req)
175175

176176
int neu_json_decode_update_user_req(char *buf, neu_json_password_req_t **result)
177177
{
178-
return neu_json_decode_password_req(buf, result);
178+
int ret = 0;
179+
neu_json_password_req_t *req = calloc(1, sizeof(neu_json_password_req_t));
180+
neu_json_elem_t req_elems[] = {
181+
{
182+
.name = "name",
183+
.t = NEU_JSON_STR,
184+
},
185+
{
186+
.name = "new_password",
187+
.t = NEU_JSON_STR,
188+
},
189+
};
190+
ret = neu_json_decode(buf, NEU_JSON_ELEM_SIZE(req_elems), req_elems);
191+
if (ret != 0) {
192+
goto decode_fail;
193+
}
194+
195+
req->name = req_elems[0].v.val_str;
196+
req->new_pass = req_elems[1].v.val_str;
197+
198+
*result = req;
199+
return ret;
200+
201+
decode_fail:
202+
free(req);
203+
return -1;
179204
}
180205

181206
void neu_json_decode_update_user_req_free(neu_json_password_req_t *req)
182207
{
183-
neu_json_decode_password_req_free(req);
208+
if (req) {
209+
free(req->name);
210+
free(req->new_pass);
211+
free(req);
212+
}
184213
}
185214

186215
int neu_json_decode_delete_user_req(char * buf,

0 commit comments

Comments
 (0)