Skip to content

added 200 status api check product, test, and finding#114

Merged
devGregA merged 6 commits intoDefectDojo:masterfrom
devGregA:master
Sep 10, 2016
Merged

added 200 status api check product, test, and finding#114
devGregA merged 6 commits intoDefectDojo:masterfrom
devGregA:master

Conversation

@devGregA
Copy link
Copy Markdown
Contributor

@devGregA devGregA commented Sep 9, 2016

No description provided.

@devGregA
Copy link
Copy Markdown
Contributor Author

devGregA commented Sep 9, 2016

@aaronweaver @mtesauro any idea what is going wrong here guys?

https://travis-ci.org/OWASP/django-DefectDojo/builds/158878615

screen shot 2016-09-09 at 6 38 25 pm

@aaronweaver
Copy link
Copy Markdown
Contributor

It looks like the environment variables in Travis aren't set but the screenshot you showed has them set. If I look in prior builds I see Travis listing the environment variables (build #6) but in build #8 I don't see them.

build__6_-owasp_django-defectdojo-_travis_ci

build__8_-owasp_django-defectdojo-_travis_ci

@mtesauro
Copy link
Copy Markdown
Contributor

+1 on what Aaron said.

I was about to write that but he's quicker then me. The admin user and password are blank since those environment variables are not set and the test cannot log in whatever values it has for the admin creds.

@devGregA devGregA merged commit 76e9981 into DefectDojo:master Sep 10, 2016
@devGregA
Copy link
Copy Markdown
Contributor Author

Figured it out. Apparently env variables from the repo are only loaded for commits not PRs... I added export lines to 'before install'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants