svrjs/svrjs-nextjs-website
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix vulnerability deletion API. #11

Merged
DorianNiemiecSVRJS merged 1 commit from main into main 2024-08-09 13:46:28 +02:00
Conversation 2 Commits 1 Files changed 1 +2 -2
DorianNiemiecSVRJS commented 2024-08-09 13:45:28 +02:00 (Migrated from github.com)
Copy link

When I tried to delete a vulnerability from an administration panel, there was a 404 HTTP error. When I checked developer tools, there was a "Vulnerability not found" message. When I checked the source code, other API endpoints involving vulnerabilities use "downloadsDatabase" database, but the /api/delete/vulnerability/[id] API endpoint doesn't implicitly use "downloadsDatabase" database, just a database from the connection string, which may not match the "downloadsDatabase" database. So I probably fixed that by configuring MongoDB driver in the /api/delete/vulnerability/[id] API endpoint to use "downloadsDatabase" database.

When I tried to delete a vulnerability from an administration panel, there was a 404 HTTP error. When I checked developer tools, there was a "Vulnerability not found" message. When I checked the source code, other API endpoints involving vulnerabilities use "downloadsDatabase" database, but the /api/delete/vulnerability/[id] API endpoint doesn't implicitly use "downloadsDatabase" database, just a database from the connection string, which may not match the "downloadsDatabase" database. So I probably fixed that by configuring MongoDB driver in the /api/delete/vulnerability/[id] API endpoint to use "downloadsDatabase" database.
Proxyy587 (Migrated from github.com) reviewed 2024-08-09 13:45:28 +02:00
vercel[bot] commented 2024-08-09 13:45:32 +02:00 (Migrated from github.com)
Copy link

@DorianNiemiecSVRJS is attempting to deploy a commit to the proxyy587's projects Team on Vercel.

A member of the Team first needs to authorize it.

@DorianNiemiecSVRJS is attempting to deploy a commit to the **proxyy587's projects** Team on [Vercel](https://vercel.com). A member of the Team first needs to [authorize it](https://vercel.com/git/authorize?team=proxyy587's%20projects&type=github&job=%7B%22headInfo%22%3A%7B%22sha%22%3A%2268e44fafc75aa9447901f729a7e83956e48e6e33%22%7D%2C%22id%22%3A%22QmVWj6GWBfbG4aLvHwV8GNvi2kGjHokyDVpJeYHTVANzHK%22%2C%22org%22%3A%22Proxyy587%22%2C%22prId%22%3A11%2C%22repo%22%3A%22new-svrjs%22%7D).
netlify[bot] commented 2024-08-09 13:45:41 +02:00 (Migrated from github.com)
Copy link

Deploy Preview for svrjs ready!

Name Link
Latest commit 68e44fafc7
Latest deploy log https://app.netlify.com/sites/svrjs/deploys/66b6015a41eb6e00083d8e05
Deploy Preview https://deploy-preview-11--svrjs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse		 1 paths audited
Performance: 82
Accessibility: 98
Best Practices: 83
SEO: 92
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

### <span aria-hidden="true">βœ…</span> Deploy Preview for *svrjs* ready! | Name | Link | |:-:|------------------------| |<span aria-hidden="true">πŸ”¨</span> Latest commit | 68e44fafc75aa9447901f729a7e83956e48e6e33 | |<span aria-hidden="true">πŸ”</span> Latest deploy log | https://app.netlify.com/sites/svrjs/deploys/66b6015a41eb6e00083d8e05 | |<span aria-hidden="true">😎</span> Deploy Preview | [https://deploy-preview-11--svrjs.netlify.app](https://deploy-preview-11--svrjs.netlify.app) | |<span aria-hidden="true">πŸ“±</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br />![QR Code](https://app.netlify.com/qr-code/eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1cmwiOiJodHRwczovL2RlcGxveS1wcmV2aWV3LTExLS1zdnJqcy5uZXRsaWZ5LmFwcCJ9.2nhBTxMp74J-hQN82S_CscMIo6DOpwbxjHwhNbADSJI)<br /><br />_Use your smartphone camera to open QR code link._</details> | |<span aria-hidden="true">![Lighthouse](https://app.netlify.com/public/external-usage/git-comments/lighthouse-sm.png)</span><br />Lighthouse | 1 paths audited<br />**Performance**: 82<br />**Accessibility**: 98<br />**Best Practices**: 83<br />**SEO**: 92<br />**PWA**: -<br />[View the detailed breakdown and full score reports](https://app.netlify.com/sites/svrjs/deploys/66b6015a41eb6e00083d8e05) | --- _To edit notification comments on pull requests, go to your [Netlify site configuration](https://app.netlify.com/sites/svrjs/configuration/deploys#deploy-notifications)._
Sign in to join this conversation.
Reviewers
No reviewers
Proxyy587
Labels
Clear labels
  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: svrjs/svrjs-nextjs-website#11
No description provided.
Delete branch "main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?