tylerlight071/nexus-pet-portal

An Animal Adoption Management System built in Python that allows a Rescue Centre to manage the animals in their shelter in a variety of ways.

💻 Tech Stack:

📊 GitHub Stats:

📚 Random Quote:

🤣 Random Meme:

nexus-pet-portal's Issues

Change View Animal Profile to match Current User

Is your feature request related to a problem? Please describe.
The use of tkinter for the animal profile is jarring as it does not match the rest of the script in its current state and should ideally match the current user output of the Update User Information in the ADMIN Dashboard

Move Sort and Filter into the Search For Animals output

Is your feature request related to a problem? Please describe.
Move Sort and Filter into the Search For Animals, View Animals & Modify Animal Database output rather than in main options as it is pointless to have due to the user not being able to see a table in this view whereas in the search, the possibility to have multiple animals of the same is likely which is the intended use

Connection to MongoDB fails on first time inputting URI before logging in

Describe the bug
When the user first runs the program and enters the URI, it says the connection is successful or not as intended, however when the user attempts to log in without rebooting the app which is expected, it does not connect.

Expected behaviour
User is expected to enter their URI and then proceed to login without any issues

User can set any value to gender in edit animal entries

Describe the bug
User can set any value to gender in edit animal entries rather than the Male & Female the program requires

No username or password --> Incorrect password message displayed

Describe the bug
If the user does not input a username or password, the error message for an incorrect password is displayed and the attempts is increased by 1

Expected behaviour
Error message that says "Username not found" should be displayed

ValueError when enter empty value into ADMIN update user level

Describe the bug
Error when user presses enter when value is empty due to no error handling to counter that

Sudo Login --> No password, no error

Describe the bug
If the user is >= level 2 and does not input a password into the sudo login, no error is shown, the screen is not cleared and the sudo login is looped

Expected behaviour
Error message is displayed, screen cleared and sudo looped

Delete Animal --> No table when returned to option menu

Describe the bug
No animal table is displayed when returned to the option menu after deleting an animal

Notifications

This is a temporary thing that will be tested to see its use but could be used in regard to the staff portal to remind the user that they have events coming up but it is likely to be implemented properly in a future release, but for now will remain with the placeholder notifications to see how it may work as a concept idea

Filter Animals does not accept Male, only male, nor Female, only female

Describe the bug
The filter animals does not accept Male or Female as a valid value, only male or female

To Reproduce
Steps to reproduce the behaviour:

1. Go into view animals
2. Enter filter
3. Input Male or Female and the error arises

Expected behaviour
Supposed to display the animals with the labels Male and Female just as using male and female does

Username not found

Move the username not found check to after the user inputs their username to make the process more streamlined

Config.py does not work in exe

Describe the bug
When user inputs new URI the exe does not use that URI but instead the default that is bundled. To fix, changing from a .py to JSON so it can dynamically update and pull information as needed

ADMIN commands looking for users.json

Describe the bug
ADMIN commands:

• update user information
• change user password
• delete user

are all looking to load and update users.json but moved to database so longer needed

clear_screen_and_print_animals( ) was given 1 argument, expecting 0

Describe the bug
As in the title, the function is being given too many arguments and must be trimmed to meet the new style of the application and is as simple as removing the argument 'animals' from the 'handle_x_result functions and the clear_screen_and_print_animlas( ) function in the view_animals.py script

Clear Filters

Is your feature request related to a problem? Please describe.
A way to clear the filters without exiting and re-entering the view animals --> this allows the user to stay on the same page and refresh any previous filters

Describe the solution you'd like
An addition to the filters tab that simply allows the user to clear all the current filters

ADMIN Sudo --> Screen not cleared after verification

Describe the bug
If the user's ADMIN account is verified through the ADMIN Sudo Login, the screen is not cleared, presenting the new output underneath the User Verified... text rather than clearing the screen, then presenting the new text

Upload image error

Describe the bug
Clear error where save_data() is missing the argument filename

To Reproduce
Steps to reproduce the behaviour:

1. Open the animal's profile
2. Upload a file
3. Error occurs

Expected behaviour
Image is shown in the profile after refreshing the screen

Fix code scanning alert - Use of a broken or weak cryptographic hashing algorithm on sensitive data

Tracking issue for:

• https://github.com/tylerlight071/FurEver-Friends/security/code-scanning/2

Dependabot found this issue and I will be looking into a stronger way before the first version as this is a very basic way to hash data to get the feel of how I could do it and how it would work. Will have a much stronger way in future versions either with or before the V1.0 release

Invalid Input message --> does not clear screen

Describe the bug
If the user does not input anything into the search animal profile, prints error message, but does not clear screen nor print the animal table

Script does not move on when user enters full address into the customer_adoption_form.py

Describe the bug
When the user enters the full address, it does not move onto the rest of the form but instead goes back to the options menu. It does not mark the animal as adopted nor save the client's information in the database.

Expected behaviour
Supposed to require second consent acknowledgement before proceeding to mark the animal as adopted and save the customer information to the database

Fix code scanning alert - Use of a broken or weak cryptographic hashing algorithm on sensitive data

Tracking issue for:

• https://github.com/tylerlight071/FurEver-Friends/security/code-scanning/3

Dependabot found this issue and I will be looking into a stronger way before the first version as this is a very basic way to hash data to get the feel of how I could do it and how it would work. Will have a much stronger way in future versions either with or before the V1.0 release

Fix code scanning alert - Use of a broken or weak cryptographic hashing algorithm on sensitive data

Tracking issue for:

• https://github.com/tylerlight071/FurEver-Friends/security/code-scanning/4

Dependabot found this issue and I will be looking into a stronger way before the first version as this is a very basic way to hash data to get the feel of how I could do it and how it would work. Will have a much stronger way in future versions either with or before the V1.0 release

Attempts remaining does not show

Describe the bug
When the user inputs the incorrect password, it correctly removes an attempt from the user but the screen is cleared too fast to display the message, rendering it unreadable.

User kicked out of Client Database

Describe the bug
If the user enters a "coming soon" section, they are kicked out the database rather than looped back into the options menu

Attempts remaining & No message for username not found --> Sudo User

Describe the bug
If the user enters the the wrong username, or no password, they are told that both the username and password are required rather than the message to say that username was not found if wrong username and both fields are required if the user misses one, or both fields.

Also when the user does this, they are also not shown the attempts remaining text but are however kicked out the application after 2 failed attempts as intended

Fix code scanning alert - Starting a process with a shell, possible injection detected, security issue.

This is a minor issue that holds no risk as there is no user input here, however I will implement something that potentially fixes this issue for peace of mind

Tracking issue for:

• https://github.com/tylerlight071/Nexus-Pet-Portal/security/code-scanning/6

Animal table is shown when user exits any of the animal database options

Describe the bug
When the user exits the animal database options ie, Search, Sort/Filter etc, the animal table is printed rather than just the options as it should

-> This is due to unfinished modification of the script in regards to UI modifications and will be fixed promptly along with the new tabulate interface

Fix code scanning alert - Possible SQL injection vector through string-based query construction.

This is a issue that keeps popping up and with research it appears to not be an issue, however am working on possible fix and will monitor to either modify or remove fix. This issue seems to only be with the else statement where it logs the deleted animal

Tracking issue for:

• https://github.com/tylerlight071/Nexus-Pet-Portal/security/code-scanning/28

Audit Logs

Is your feature request related to a problem? Please describe.
Implement a way for the user to see their audit logs based on the date, time, user and/or activity in the ADMIN panel

ADMIN User Password reset

Is your feature request related to a problem? Please describe.
ADMIN user has no way to reset their own password should it be compromised and or if they lose or forget the password. A solution is being planned and looked into with the mentality of emails or security questions being present.

• User can currently delete the config file or delete the users collection in their database and start again

Notifications not accessible with level 2+

Describe the bug
If the user tries to enter the notifications and is level 2+, it says invalid input

Incorrect password for Sudo User still allows access

Describe the bug
If the user inputs the incorrect password into the Sudo Login section, it tells the user it is incorrect yet still allows them access rather than make the user try again and remove an attempt

Filter Animals does not accept multiple queries

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behaviour:

1. In view animals, go into the filter
2. Attempt to input two or more queries
3. Message, no animals found with that criteria is displayed

Expected behaviour
Display all animals that meet the criteria displaying only those with ALL the inputted queries

Fix code scanning alert - Use of a broken or weak cryptographic hashing algorithm on sensitive data

Tracking issue for:

• https://github.com/tylerlight071/FurEver-Friends/security/code-scanning/1

Dependabot found this issue and I will be looking into a stronger way before the first version as this is a very basic way to hash data to get the feel of how I could do it and how it would work. Will have a much stronger way in future versions either with or before the V1.0 release

Table is not optimized should the index of the table exceed 9

Describe the bug
If the amount of animals in the table exceeds 9, the table from 10 onwards are moved to the right by one. Probably do this for every extra digit