-
-
Notifications
You must be signed in to change notification settings - Fork 363
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
Change from single user (config) to database based multi user system. #1165
Open
fnkbsi
wants to merge
31
commits into
steve-community:master
Choose a base branch
from
fnkbsi:MultiUsers
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
ae0d9d2
Change from single user (config) to database based multi user system.
61e3f00
style checks
7a15857
delete noAccessController to rename
9d84a5a
add renamed NoAccessController
1359cc8
style check
cc3db61
Removed out commented code; cleaned the import areas, removed useless…
4f51fc3
Increased spring-boot version to 3.1.0
45d9ab3
Revert Commit, because not Java 11 compatible
7aa3bd6
adding missing newline at the file end (style check)
6564cdc
WebUserController: correct return url if passwort is blank
edd2365
BeanConfiguration: added @Autowired to HikariDataSource and @Bean to …
778318f
SecurityConfiguration: changed DataSource to HikariDataSource and add…
fa7a2fc
pom.xml: revomed dependency of org.springframework.boot and moved dep…
54e7198
BeanConfiguration: initDataSource returns the new HikariDataSourcedir…
98b9ce8
BeanConfiguration: renamed initDataSource() --> dataSource(); removed…
040878c
NoAccessController: removed the RequestMethod, so all requests can be…
647ca1c
SecurityConfiguration: adapt SecurityFilterChain, so Usesr have "read"
42980df
Merge branch 'steve-community:master' into MultiUsers
fnkbsi 0236b55
Merge branch 'steve-community:master' into MultiUsers
fnkbsi 0276925
Rename db migration script to avoid conflict with master. Included if…
bd62f19
Rename db migration script removed "if not exist" clauses because of …
2f67258
Merge db migration script V1_0_4 from master
6fde7dc
Merge branch 'steve-community:master' into MultiUsers
fnkbsi d2273f7
Merge branch 'steve-community:master' into MultiUsers
fnkbsi 4a7a616
Merge branch 'steve-community:master' into MultiUsers
fnkbsi ab84315
Merge branch 'steve-community:master' into MultiUsers
fnkbsi ed4e602
Merge branch 'steve-community:master' into MultiUsers
fnkbsi 6a6c96a
Merge branch 'steve-community:master' into MultiUsers
fnkbsi a3d6d16
Merge branch 'steve-community:master' into MultiUsers
fnkbsi a3f2fd2
Merge branch 'steve-community:master' into MultiUsers
fnkbsi 291f683
Merge branch 'steve-community:master' into MultiUsers
fnkbsi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Increased spring-boot version to 3.1.0
- Loading branch information
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
It looks the 2 new dependencies are not used. They should be removed IMO.
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.
Both dependencies used in SecurityConfiguration.java
by "import org.springframework.boot.jdbc.DataSourceBuilder;" (line 55)
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.
steve is not using spring boot though. it is a spring app, not a spring boot app.
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.
@fnkbsi steve uses jooq for the data access. You should be able to replace spring-jdbc by that. Then no need to datasourcebuilder.
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.
@goekay any feedback about the feature itself?
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.
i find the feature very valuable, but need time to to review the code. at a glance, i find that the implementation needs some tidying up which is already happening. thanks @fnkbsi