Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Deploy on public server #253

Closed
4 tasks done
pombredanne opened this issue Sep 10, 2020 · 17 comments
Closed
4 tasks done

Deploy on public server #253

pombredanne opened this issue Sep 10, 2020 · 17 comments

Comments

@pombredanne
Copy link
Member

pombredanne commented Sep 10, 2020

  • Decide on a DNS domain name and acquire name
  • Provision server (Philippe), possibly with GCP credits at least for the initial DB creation
  • create deploy and backup scripts
  • deploy proper
@pombredanne pombredanne created this issue from a note in VulnerableCode (To do) Sep 10, 2020
@pombredanne
Copy link
Member Author

@edoardolanzini ping, FYI

@sbs2001
Copy link
Collaborator

sbs2001 commented Feb 7, 2021

The temp domain for now would be https://vulnerablecode.aboutcode.org

@pombredanne
Copy link
Member Author

@sbs2001 I registered for now vulnerabilitydb.org which is simple and generic

@pombredanne
Copy link
Member Author

and I provisioned a dedicated server

@pombredanne pombredanne moved this from To do to In progress in VulnerableCode Aug 24, 2021
@pombredanne pombredanne moved this from In progress to Backlog in VulnerableCode Feb 2, 2022
@pombredanne pombredanne added this to the v30.0 milestone Aug 2, 2022
@pombredanne
Copy link
Member Author

This has been deployed (with a simple PW until final release 30.0.0 is tagged) at https://public.vulnerablecode.io/ 🎉

@sschuberth
Copy link

This has been deployed (with a simple PW until final release 30.0.0 is tagged) at https://public.vulnerablecode.io/ 🎉

Meanwhile release 30.2.0 is tagged. Is there a way we could use the public instance in an ORT example pipeline without a password now, @pombredanne?

@pombredanne
Copy link
Member Author

@sschuberth there is a new release coming up and we added API doc and self registration for an API key
This is may be released this week.

@sschuberth
Copy link

self registration for an API key

Would you be OK with exposing an API key for ORT in a public example ORT pipeline that leverages VulnerableCode, @pombredanne?

@pombredanne
Copy link
Member Author

And this is now live at https://public.vulnerablecode.io/

@pombredanne
Copy link
Member Author

Would you be OK with exposing an API key for ORT in a public example ORT pipeline that leverages VulnerableCode,

Your call. An API key is like a password... and API calls should be throttled. Getting an API key should be easy enough

@sschuberth
Copy link

And this is now live at https://public.vulnerablecode.io/

This is what I get when requesting an API key:

image

@pombredanne
Copy link
Member Author

@sschuberth This is now fixed! Sorry for the noise!

@sschuberth
Copy link

I've got my API key now, but unfortunately I'm getting "Authentication credentials were not provided." for any API call that I tried so far despite an "Authorization" header with my token being present.

@sschuberth
Copy link

despite an "Authorization" header with my token being present.

Ah, the value of that header field needs to start with the literal word "Token" before the token's value!

Could you fix the cURL code generated at https://public.vulnerablecode.io/api/docs/ to include that?

@pombredanne pombredanne reopened this Nov 10, 2022
@pombredanne
Copy link
Member Author

Let me reopen this issue
In the OpenAPI at https://public.vulnerablecode.io/api/docs/ if I click on Authorize, the popup states:

Available authorizations

tokenAuth (apiKey)

Token-based authentication with required prefix "Token"

Name: Authorization

In: header
Value:

Where do you think we could make this more obvious?

@sschuberth
Copy link

sschuberth commented Nov 10, 2022

Where do you think we could make this more obvious?

Ah, my fault. In this dialog

image

I was overlooking the with required prefix "Token" part and just pasted the hex value. If the "Token " prefix is included here, it also shows up in the cURL command line snippets. All a bit error-prone because unusual IMO, but actually correctly documented.

So, thanks, I think we can close this again!

@TG1999
Copy link
Member

TG1999 commented Dec 7, 2022

Thanks @sschuberth , closing this!

@TG1999 TG1999 closed this as completed Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
VulnerableCode
  
Backlog
Development

No branches or pull requests

4 participants