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

Cleanup gsad #1265

Merged
merged 7 commits into from
Apr 9, 2019
Merged

Cleanup gsad #1265

merged 7 commits into from
Apr 9, 2019

Conversation

bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Apr 9, 2019

  • Drop obsolete CLI arguments from gsad
  • Fix handling of git revision for gsad build
  • Update copyright in printed with --version

Related to #1258

Checklist:

Guest user support is implemented in GSA now. There is no need for extra
code in gsad anymore.

This commit removes the following CLI arguments:

* --http-guest-chart-frame-opts
* --http-guest-chart-csp
* --guest-username
* --guest-password
This is a left over from GSA 7. The commit removed the --login-lable CLI
argument.
Allow for easier handling of the copyright displayed with the --version
switch.
If gsa isn't build from a git clone don't set the GIT_REVISION variable.
The git revision is included in the GSAD_VERSION variable automatically.
There is no need to set a define, generate an extra header or something
else.
@bjoernricks bjoernricks marked this pull request as ready for review April 9, 2019 09:13
@bjoernricks bjoernricks requested a review from a team April 9, 2019 09:13
@bjoernricks bjoernricks merged commit e4efd14 into greenbone:gsa-8.0 Apr 9, 2019
@bjoernricks bjoernricks deleted the cleanup-gsad branch April 9, 2019 10:04
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.

2 participants