Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Hotfix/man choose originherd #578

Merged
merged 5 commits into from
Nov 28, 2022
Merged

Hotfix/man choose originherd #578

merged 5 commits into from
Nov 28, 2022

Conversation

jhagberg
Copy link
Collaborator

Diverse fix. Fix so manager can register a litter in any herd other than the current active herd of the mother.

github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 28, 2022
View reviewed changes
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 28, 2022
View reviewed changes
app/utils/data_access.py
Comment on lines +869 to +870
f"{user.username},{individual.number},"
f"certificate,{getattr(individual,'certificate')},'None',"

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text.
app/utils/data_access.py
Comment on lines +879 to +880
f"{user.username},{individual.number},"
f"digital_certificate,{getattr(individual,'digital_certificate')},'None',"

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information

This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text. This expression logs [sensitive data (certificate)](1) as clear text.
@jhagberg jhagberg merged commit f2b241a into develop Nov 28, 2022
@jhagberg jhagberg deleted the hotfix/man-choose-originherd branch November 28, 2022 16:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jhagberg