Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Revert "Revert Auth service back to Spring Boot 1.x"

This reverts commit d7de12a691485f3412f0342932693ed406151186.

  1. … 43 more files in changeset.
Revert Auth service back to Spring Boot 1.x

To see if it is the cause of the Superset issue.

  1. … 43 more files in changeset.
OLMIS-6761 Upgrade Spring Boot to 2.2.2
OLMIS-6761 Upgrade Spring Boot to 2.2.2
OLMIS-6761 Update Spring Boot version to 2.x

Using Boot version 2.2.2, as it is relatively recent.

* Flyway is at 6.0.8, new mechanism for loading Spring Security for OAuth2, and new versions for REST Assured, RAML tester, RAML parser, PowerMock (so tests will pass).

* Spring application properties for Flyway has changed.

* Fix repository method signatures (findOne is now findById, etc.); additionally they return Optional.

* Spring Security 5 has changed encoding of password storage to DelegatingPasswordEncoder. To be backwards-compatible, currently returning NoOpPasswordEncoder as default. Will probably need to migrate all passwords to new encoding and use a better encoder/decoder.

* Fix unit tests.

* Flyway is now at v6, and Java callback mechanism has changed to a general handle() method.

* Fix integration tests (including simplifying TokenIntegrationTest).

* Integration tests allow bean definition overriding, as it is not enabled by default now in Spring Boot 2.x.

* API definitions require "Keep-Alive" header for web integration tests.

  1. … 33 more files in changeset.
OLMIS-4295: updated checkstyle to new google style
OLMIS-4295: updated checkstyle to new google style
OLMIS-4295: updated checkstyle to new google style

fixed LICENCE header

fixed import order

fixed minor code style issues

added missing periods in javadocs

  1. … 157 more files in changeset.
Yes, I'm wondering what is better. I remember reading about always using exists if posibble, even if you have to retrieve whole object in some Performance Guidance or something in read the docs.

Yes, I'm wondering what is better. I remember reading about always using exists if posibble, even if you have to retrieve whole object in some Performance Guidance or something in read the docs.

moved

moved

OLMIS-4986: added missing unit tests

  1. … 2 more files in changeset.
but I still need it if it exists

but I still need it if it exists

Yes

Yes

We could use exists here to not get whole object first.

We could use exists here to not get whole object first.

This fix is applied to changes made after latest release?

This fix is applied to changes made after latest release?

I feel like this part should be in service and tested with unit tests

I feel like this part should be in service and tested with unit tests

OLMIS-4986: Fixed problem with saving auth user
OLMIS-4986: Fixed problem with saving auth user
OLMIS-4986: Changed the user resource structure
OLMIS-4986: Changed the user resource structure
moved to user contact details notification service because there are email addresses

moved to user contact details notification service because there are email addresses

we don't do that here meme

we don't do that here meme

OLMIS-4986: Changed the user resource structure

    • -0
    • +85
    ./ConditionalUuidGeneratorTest.java
  1. … 26 more files in changeset.