-
Notifications
You must be signed in to change notification settings - Fork 9
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
Feature/clan6 199 remove cogboard init task #113
Feature/clan6 199 remove cogboard init task #113
Conversation
Configuration files moved to resources `widget1.json` deleted `admins.conf` -> `admins.json` `.gitignore` updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please consider my propositions
cogboard-app/src/main/kotlin/com/cognifide/cogboard/config/service/CredentialsService.kt
Outdated
Show resolved
Hide resolved
|
||
fun loadConfig(): JsonObject = storage.loadConfig() | ||
fun loadConfig(): JsonObject = VolumeStorageFactory.endpoints().loadConfig() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
similar as my previous comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
|
||
object AdminsValidator : Validator { | ||
override fun validate(config: String): Boolean { | ||
if (File("/data/admins.json").exists()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this validation makes no sense now as i will be always created before code reaches this point... Lets validate if at least one admin is defined with password
cogboard-app/src/main/kotlin/com/cognifide/cogboard/security/LoginHandler.kt
Outdated
Show resolved
Hide resolved
cogboard-app/src/main/kotlin/com/cognifide/cogboard/storage/StorageInitializer.kt
Outdated
Show resolved
Hide resolved
cogboard-app/src/main/kotlin/com/cognifide/cogboard/storage/StorageInitializer.kt
Outdated
Show resolved
Hide resolved
…emove-cogboardInit-task # Conflicts: # build.gradle.kts
…Validator not done yet.
…github.com/Cognifide/cogboard into feature/CLAN6-199-Remove-cogboardInit-task
No description provided.