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

Consistent settings #1774

Merged
merged 6 commits into from
Nov 12, 2019
Merged

Consistent settings #1774

merged 6 commits into from
Nov 12, 2019

Conversation

bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Nov 12, 2019

Use consistent settings and setting names in GSA.

Checklist:

Use camel case for log level.
Recognize the old loglevel config variable and mark it as deprecated.
Use camel case for protocol doc url setting.
Use apiServer and apiProtocol instead because both settings are used for
a connection to the web server providing the API. Static content is
still loaded from the current server.
Instead of gmp.reloadInterval the value MUST BE used from
gmp.settings.reloadInterval always.

Also it was already obsolete in withEntityContainer because the setting
has been moved to the Reload component already.
@bjoernricks bjoernricks marked this pull request as ready for review November 12, 2019 11:22
@bjoernricks bjoernricks requested a review from a team November 12, 2019 11:22
@codecov
Copy link

codecov bot commented Nov 12, 2019

Codecov Report

Merging #1774 into gsa-8.0 will increase coverage by 0.03%.
The diff coverage is 87.5%.

Impacted file tree graph

@@             Coverage Diff             @@
##           gsa-8.0    #1774      +/-   ##
===========================================
+ Coverage    39.98%   40.01%   +0.03%     
===========================================
  Files          965      965              
  Lines        22135    22150      +15     
  Branches      6270     6276       +6     
===========================================
+ Hits          8850     8863      +13     
- Misses       12027    12029       +2     
  Partials      1258     1258
Impacted Files Coverage Δ
gsa/src/gmp/gmp.js 80% <ø> (+2.38%) ⬆️
gsa/src/web/entity/withEntityContainer.js 39.28% <ø> (ø) ⬆️
gsa/src/web/app.js 0% <0%> (ø) ⬆️
gsa/src/web/components/link/protocoldoclink.js 100% <100%> (ø) ⬆️
gsa/src/gmp/http/gmp.js 62.5% <100%> (ø) ⬆️
gsa/src/gmp/log.js 100% <100%> (ø) ⬆️
gsa/src/gmp/gmpsettings.js 95.4% <86.66%> (-4.6%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe70e90...6bf377d. Read the comment docs.

@bjoernricks bjoernricks merged commit 5e38ef7 into greenbone:gsa-8.0 Nov 12, 2019
@bjoernricks bjoernricks deleted the consistent-settings branch November 12, 2019 11:30
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