Removed unused legacy forwards.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

The JWT is now read from the Authorization header:
	Authorization: Bearer <JWT>
The Arsnova-Auth-Token header is no longer supported.

Using a standard header simplifies integration of the API with other
services and HTTP/REST libraries and tools.

RFC 6750:
https://tools.ietf.org/html/rfc6750

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

We do not use any data sources at the moment and the auto config causes
problems when the application is deployed to Tomcat.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

Methods annotated with @Timed now use Micrometers annotation. The @Gauge
annotation has been remove without replacement for now.

While configuration for the use of Micrometer's TimedAspect has been
prepared, it is not yet active because of compatibility issues with
compile-time weaving.
See https://github.com/micrometer-metrics/micrometer/issues/1149.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

The new endpoint provide metrics including information about application
health, environment, configuration, beans, caches and performance via
Spring Actuator. Additional custom endpoints for version info and
statistics are available.

Content type `application/json` may not be set for the `Accept` header
of requests to the management API. Otherwise, only a JSON stub is
returned. `application/vnd.spring-boot.actuator.v2+json` or `*/*` should
be used instead.

To allow monitoring of metrics, a Prometheus endpoint is provided via
Micrometer.

Web API documentation:
https://docs.spring.io/spring-boot/docs/2.1.x/actuator-api/html/

Daniel Gerhardt authored 5 years ago and Tom Käsler committed 5 years ago

The Nashorn ScriptEngine is deprecated since JDK 11 as described by
JEP 335. We now include GraalJS as a replacement.

Related docs:
* https://github.com/graalvm/graaljs.
* https://openjdk.java.net/jeps/174.
* https://openjdk.java.net/jeps/335.

Validation is performed before entities are persisted using Hibernate
Validator, the reference implementation for Bean Validation 2.0
(JSR 380).

Refs #23.

The location of `application.yml` can now be set via environment
variable `ARSNOVA_CONFIG_DIR` or Java system property
`arsnova.config-dir`.

For most of the configuration property classes are now used instead of
using property injection via @Value.

* Removed obsolete commented-out code.
* Fixed indentation of commented out code.
* Added FIXME comments where old code still has to be migrated.

Rules:
* (naming) ParameterName
* (naming) LocalVariableName
* (naming) MemberName
* (naming) InterfaceTypeParameterName
* (naming) AbbreviationAsWordInName
* (modifier) ModifierOrder
* (blocks) NeedBraces
* (coding) OneStatementPerLine
* (coding) MultipleVariableDeclarations
* (coding) VariableDeclarationUsageDistance
* (coding) MissingSwitchDefault
* (coding) OverloadMethodsDeclarationOrder
* (extension) RightCurlySame
* (misc) ArrayTypeStyle

The following conventions have been applied:
* Rule: (sizes) LineLength

The following conventions have been applied:
* Fix indentation levels
  Rules: (indentation) Indentation, (regexp) RegexpSinglelineJava
* Always use operators at the beginning of a new line if there is a line
  break.
  Rule: (whitespace) OperatorWrap
* Always use empty lines between method declarations.
  Rule: (whitespace) EmptyLineSeparator
* Always use new line between annotation and field/method declaration.
  Rule: (extension) AnnotationLocationMostCases
* Additional rules:
    * (whitespace) ParenPad
    * (whitespace) GenericWhitespace
    * (blocks) LeftCurly
    * (extension) RightCurlyAlone

The following conventions have been applied:
* Import order: static packages -> other packages -> ARSnova packages
  Rule: (imports) CustomImportOrder
* New line between import package groups
  Rule: (imports) CustomImportOrder
* All packages are imported explicitly (no wildcard imports)
  Rule: (imports) AvoidStarImport
* New line between license comment and package line
  Rule: (whitespace) EmptyLineSeparator

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

This also fixes the read permission room owners. They now have read
access to closed rooms.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

The `/room/find` endpoint has been extended to support querying rooms by
moderator user ID. It can be used in combination with the `ownerId`
property.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

Moderators can assist a room owner in managing the room. This commit
adds API and persistence support for moderators.

An error was logged before destroying UserServiceImpl class without any
reason.

An error was logged before destroying UserService class without any
reason.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

Authenticated users can read `loginId` and `authProvider` properties.
A FindQueryService has been added to lookup UserProfiles by `loginId`.

Christoph Thelen authored 5 years ago and Daniel Gerhardt committed 5 years ago

Two key changes:

  1. Add a factory method to create the EntityService with the
     DummyEntity, so that all other test cases can initialize
     this Bean.  This solves the IllegalStateException reported
     in this ticket.
  2. Use @Autowire instead of @MockBean to use the new factory
     method.  That resolves the known issue that the caching test
     case does not work because of different RoomRepositories being
     created for the PermissionEvaluatorAdvice and the EntityService.

This makes all tests runnable again.  My current theory is that due to
the fact that we have at least two EntityServices (one created by
the DefaultEntityServiceImplTest, and now one by this test).  The
@MockBean annotation could be throwing out the previous EntityService,
since according to documentation, it replaces existing Beans with the
same type.  This could lead to a situation where, initially, the
PermissionEvaluatorAdvice and the DefaultEntityService share the same
RoomRepository.  But after the @MockBean replacement occurs, the
EntityService is recreated with a different RoomRepository, while the
Advice keeps its initial RoomRepository, leading to testing failures
in the testCaching test case.

Christoph Thelen authored 5 years ago and Daniel Gerhardt committed 5 years ago

This solves the IllegalStateExceptions occurring for most of the other
test cases.

However, we now have the old problem that different Repositories are
injected into objects. In my case, the testCaching case of the
DefaultEntityServiceImplTest stops working, because it gets a different
RoomRepository than the ApplicationPermissionEvaluator.

This behavior is caused by the introduction of the additional profile.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

See commit f620b9e7.

A JsonView for serialization is applied based on the 'view' query
parameter.

The following values are currently supported:
* admin (View.Admin.class)
* owner (View.Owner.class)
* <unset> (Public.Admin.class)