mirror of
https://github.com/norandom/OpenUltraSAST.git
synced 2026-10-16 03:49:26 +00:00
Marius CiepluchandClaude Opus 5
c2d9c4ad01
Eight of the ten vendored access-control rows had no named handler at all, so every one of them would have scored `missing_context` forever rather than as a detector miss. After the re-harvest nine are named and the tenth states its reason. The handler-context mode now carries the class its framework registers, multi-line decorators, and commented-out guards -- `#@token_required` above a ThreatByte handler is the entire bug. Where the registration lives in another document (connexion's `operationId`, an express router), a recipe names `context = [...]`, the harvester writes that stanza per side, and the pair materializes both files at their repository-relative paths so the entry-point mapper sees the layout the project has. `harvest_recipe` fetches every blob before the first write and the driver takes `--report`, so a failed fetch leaves a reviewed excerpt exactly as it was. Two defects the re-harvest exposed: `.mjs` was not a known suffix, so the one pair vendored as an ES module had no file target at all; and `_python_route_access` matched auth words inside decorator prose, reading `@ns.doc(description='... without proper authorization checks')` as a guard. Recorded in the hygiene artifact: nine of the ten duplicate groups sit on both sides of the split, which the repository-level split cannot see. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> Claude-Session: https://claude.ai/code/session_01T7XD7x5KonMdmGRkp63EQG
45 lines
1.9 KiB
Python
45 lines
1.9 KiB
Python
# Provenance: erev0s/VAmPI update_password (fixed).
|
|
# repo: erev0s/VAmPI
|
|
# commit: 1713b54b601ad29582581eeda4b31fceb1319874
|
|
# parent: 1713b54b601ad29582581eeda4b31fceb1319874
|
|
# commit_url: https://github.com/erev0s/VAmPI/blob/1713b54b601ad29582581eeda4b31fceb1319874/api_views/users.py#L187
|
|
# cve:
|
|
# license: MIT
|
|
# function: update_password
|
|
# relpath: api_views/users.py
|
|
# provenance: human
|
|
# mechanism: missing_auth_guard
|
|
# upstream_start: 52
|
|
|
|
def register_user():
|
|
request_data = request.get_json()
|
|
# check if user already exists
|
|
user = User.query.filter_by(username=request_data.get('username')).first()
|
|
if not user:
|
|
try:
|
|
# validate the data are in the correct form
|
|
jsonschema.validate(request_data, register_user_schema)
|
|
if vuln and 'admin' in request_data: # User is possible to define if she/he wants to be an admin !!
|
|
if request_data['admin']:
|
|
admin = True
|
|
else:
|
|
admin = False
|
|
user = User(username=request_data['username'], password=request_data['password'],
|
|
email=request_data['email'], admin=admin)
|
|
else:
|
|
user = User(username=request_data['username'], password=request_data['password'],
|
|
email=request_data['email'])
|
|
db.session.add(user)
|
|
db.session.commit()
|
|
|
|
responseObject = {
|
|
'status': 'success',
|
|
'message': 'Successfully registered. Login to receive an auth token.'
|
|
}
|
|
|
|
return Response(json.dumps(responseObject), 200, mimetype="application/json")
|
|
except jsonschema.exceptions.ValidationError as exc:
|
|
return Response(error_message_helper(exc.message), 400, mimetype="application/json")
|
|
else:
|
|
return Response(error_message_helper("User already exists. Please Log in."), 200, mimetype="application/json")
|