diff --git a/fda-authentication-service/rest-service/src/main/java/at/tuwien/endpoints/TokenEndpoint.java b/fda-authentication-service/rest-service/src/main/java/at/tuwien/endpoints/TokenEndpoint.java index 7a7bea971b29ef7dab1303f03db4da485c74ec81..0fed6c2a5aa62ac0140b23d4e4aca73549725555 100644 --- a/fda-authentication-service/rest-service/src/main/java/at/tuwien/endpoints/TokenEndpoint.java +++ b/fda-authentication-service/rest-service/src/main/java/at/tuwien/endpoints/TokenEndpoint.java @@ -67,10 +67,7 @@ public class TokenEndpoint { TokenNotEligableException { log.debug("endpoint create developer token, principal={}", principal); /* check */ - final List<Token> tokens = tokenService.findAll(principal) - .stream() - .filter(t -> Objects.isNull(t.getDeleted())) - .collect(Collectors.toList()); + final List<Token> tokens = tokenService.findAll(principal); log.trace("found all tokens {}", tokens); if (tokens.size() >= authenticationConfig.getTokenCount()) { log.error("Failed to create token, already exceeded maximum quota of {}", authenticationConfig.getTokenCount()); diff --git a/fda-authentication-service/services/src/main/java/at/tuwien/service/impl/AuthenticationServiceImpl.java b/fda-authentication-service/services/src/main/java/at/tuwien/service/impl/AuthenticationServiceImpl.java index c259757ba94c6b68fbd155c464909e569ce0d48d..633fc469e5b266f469341cfc78c656c44c841be4 100644 --- a/fda-authentication-service/services/src/main/java/at/tuwien/service/impl/AuthenticationServiceImpl.java +++ b/fda-authentication-service/services/src/main/java/at/tuwien/service/impl/AuthenticationServiceImpl.java @@ -75,7 +75,7 @@ public class AuthenticationServiceImpl implements AuthenticationService { @Override @Transactional - public void verifyToken(String authorization) throws TokenRevokedException { + public void verifyToken(String authorization) { final String hash = authenticationMapper.authorizationToTokenHash(authorization); final Optional<Token> optional = tokenRepository.findByTokenHash(hash); if (optional.isEmpty()) { @@ -83,10 +83,6 @@ public class AuthenticationServiceImpl implements AuthenticationService { return; } final Token token = optional.get(); - if (token.getDeleted() != null) { - log.warn("Token with hash {} is marked as revoked", hash); - throw new TokenRevokedException("Token is marked as revoked"); - } token.setLastUsed(Instant.now()); tokenRepository.save(token); log.info("Updated token usage of token with hash {}", hash); diff --git a/fda-database-service/services/src/main/java/at/tuwien/service/impl/MariaDbServiceImpl.java b/fda-database-service/services/src/main/java/at/tuwien/service/impl/MariaDbServiceImpl.java index fe796dfb505b3023d63d5f381654e2c4337e7007..2bf9fc672eb0983f151d785d1f0ab357091b0321 100644 --- a/fda-database-service/services/src/main/java/at/tuwien/service/impl/MariaDbServiceImpl.java +++ b/fda-database-service/services/src/main/java/at/tuwien/service/impl/MariaDbServiceImpl.java @@ -107,7 +107,6 @@ public class MariaDbServiceImpl extends HibernateConnector implements DatabaseSe } finally { dataSource.close(); } - database.setDeleted(Instant.now()) /* method has void, only for debug logs */; /* save in metadata database */ databaseRepository.deleteById(databaseId); log.info("Deleted database with id {}", databaseId); diff --git a/fda-identifier-service/rest-service/src/test/java/at/tuwien/service/IdentifierServiceIntegrationTest.java b/fda-identifier-service/rest-service/src/test/java/at/tuwien/service/IdentifierServiceIntegrationTest.java index 49818d1e618c88587a8628561810c3c3df84985c..1954c0579eefed77985a8b1d073d9098c1d0942a 100644 --- a/fda-identifier-service/rest-service/src/test/java/at/tuwien/service/IdentifierServiceIntegrationTest.java +++ b/fda-identifier-service/rest-service/src/test/java/at/tuwien/service/IdentifierServiceIntegrationTest.java @@ -89,7 +89,6 @@ public class IdentifierServiceIntegrationTest extends BaseUnitTest { .created(IDENTIFIER_2_CREATED) .lastModified(IDENTIFIER_2_MODIFIED) .publicationYear(IDENTIFIER_2_PUBLICATION_YEAR) - .deleted(Instant.now().minus(4, ChronoUnit.MINUTES)) .publisher(IDENTIFIER_2_PUBLISHER) .type(IDENTIFIER_2_TYPE) .build()); diff --git a/fda-metadata-db/Dockerfile b/fda-metadata-db/Dockerfile index f9b7c8becec4a1223fdc83239575ae335ff071aa..32417ec6bf752371d9ba22f3fba8b02b9d08f267 100644 --- a/fda-metadata-db/Dockerfile +++ b/fda-metadata-db/Dockerfile @@ -25,17 +25,13 @@ ENV METADATA_USERNAME=root ENV METADATA_PASSWORD=dbrepo ENV MARIADB_DATABASE="${METADATA_DB}" ENV MARIADB_ROOT_PASSWORD="${METADATA_PASSWORD}" -ENV MARIADB_USERNAME="${METADATA_USERNAME}" -ENV MARIADB_PASSWORD="${METADATA_PASSWORD}" # Scripts are copied to /docker-entrypoint-initdb.d/ in docker-compose from analyze service HEALTHCHECK --interval=10s --timeout=5s --retries=12 CMD mysqladmin ping --user="$METADATA_USERNAME" --password="$METADATA_PASSWORD" --silent WORKDIR /docker-entrypoint-initdb.d -COPY ./setup-schema.sql ./1.sql -COPY ./setup-user.sh ./2.sh -RUN chmod +x ./2.sh +COPY ./setup-schema.sql ./setup-schema.sql WORKDIR /app diff --git a/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenBriefDto.java b/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenBriefDto.java index 6c228890452b6deeb465535cbb9d5e5c0135f280..cbb50605403f1009acba86ac22b56d2e335f27e0 100644 --- a/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenBriefDto.java +++ b/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenBriefDto.java @@ -40,8 +40,4 @@ public class TokenBriefDto { @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") private Instant expires; - @Schema(example = "2020-08-04 11:13:00") - @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") - private Instant deleted; - } diff --git a/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenDto.java b/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenDto.java index 6e3868f520a6126e006510a4c45c8da80b5dd08b..1cf442509c4207dd697b5afa37431a087da65ed0 100644 --- a/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenDto.java +++ b/fda-metadata-db/api/src/main/java/at/tuwien/api/auth/TokenDto.java @@ -40,10 +40,6 @@ public class TokenDto { @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") private Instant expires; - @Schema(example = "2020-08-04 11:13:00") - @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") - private Instant deleted; - @NotBlank @Schema(example = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c") private String token; diff --git a/fda-metadata-db/api/src/main/java/at/tuwien/api/database/DatabaseDto.java b/fda-metadata-db/api/src/main/java/at/tuwien/api/database/DatabaseDto.java index 9b55f61af6fb4ca98ef46be135c707f14cdf6d49..5abdf33d0b9c8211306699839f638e78cef7d5aa 100644 --- a/fda-metadata-db/api/src/main/java/at/tuwien/api/database/DatabaseDto.java +++ b/fda-metadata-db/api/src/main/java/at/tuwien/api/database/DatabaseDto.java @@ -62,8 +62,4 @@ public class DatabaseDto { @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") private Instant created; - @Schema(example = "2020-08-04 11:13:00") - @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") - private Instant deleted; - } diff --git a/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewBriefDto.java b/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewBriefDto.java index fa5346ca32d6ea26dfa3744db0bff6f5e93f49ca..2b45c7b06515ad193a71a3e865533ae347007b45 100644 --- a/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewBriefDto.java +++ b/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewBriefDto.java @@ -53,8 +53,4 @@ public class ViewBriefDto { @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") private Instant lastModified; - @Schema(example = "2020-08-04 11:13:00") - @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") - private Instant deleted; - } diff --git a/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewDto.java b/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewDto.java index 1e12924bfeeec5e7104bf6efc475f9aa757f1ce8..fcd6e9c00ae1057853aaf07d49931cd6006dc35a 100644 --- a/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewDto.java +++ b/fda-metadata-db/api/src/main/java/at/tuwien/api/database/ViewDto.java @@ -56,8 +56,4 @@ public class ViewDto { @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") private Instant lastModified; - @Schema(example = "2020-08-04 11:13:00") - @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") - private Instant deleted; - } diff --git a/fda-metadata-db/api/src/main/java/at/tuwien/api/identifier/RelatedIdentifierDto.java b/fda-metadata-db/api/src/main/java/at/tuwien/api/identifier/RelatedIdentifierDto.java index c6d5c13bf1689975c1528814c418e2bcce273818..b7ffb97c90ee374befd82606d1305efc49c2e55c 100644 --- a/fda-metadata-db/api/src/main/java/at/tuwien/api/identifier/RelatedIdentifierDto.java +++ b/fda-metadata-db/api/src/main/java/at/tuwien/api/identifier/RelatedIdentifierDto.java @@ -47,9 +47,6 @@ public class RelatedIdentifierDto { @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") private Instant lastModified; - @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSXXX", timezone = "UTC") - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/Container.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/Container.java index b8f9ecabe1ff904e91cca9f780d796f68bbf414d..25d86eee190b7c2095e35c4f0843100c43d968e1 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/Container.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/Container.java @@ -22,10 +22,8 @@ import java.util.List; @ToString @AllArgsConstructor @NoArgsConstructor -@Where(clause = "deleted is null") @EntityListeners(AuditingEntityListener.class) @EqualsAndHashCode(onlyExplicitlyIncluded = true) -@SQLDelete(sql = "update mdb_containers set deleted = NOW() where id = ?") @Table(name = "mdb_containers") public class Container { @@ -75,7 +73,4 @@ public class Container { @LastModifiedDate private Instant lastModified; - @Column - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImage.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImage.java index 07c389d693231dd1b3a2b09d4a97b95e962a9175..f6fd50d0fd4b4981f8ea1bc4c004aa85a4e188d2 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImage.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImage.java @@ -26,8 +26,8 @@ public class ContainerImage { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) public Long id; @Column(nullable = false) diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageDate.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageDate.java index 26adee03471c1de1159712a710d318f13979060b..e5624a464aa566cf6f254800ceceb163d8448f43 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageDate.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageDate.java @@ -23,8 +23,8 @@ public class ContainerImageDate { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Column(name = "iid") diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageEnvironmentItem.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageEnvironmentItem.java index 121933d4c32df6db478359126e54e4e35d77e1c5..a1abe5acad220b64922662b8d9a4c8cca58bf56a 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageEnvironmentItem.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/container/image/ContainerImageEnvironmentItem.java @@ -23,8 +23,8 @@ public class ContainerImageEnvironmentItem { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) public Long id; @Id diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/Database.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/Database.java index 190d971539b1a60ea8188f55f81dc60ba275bd02..41fe84a5286e68008a4fbdd7742bc06a856fe7f8 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/Database.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/Database.java @@ -24,9 +24,7 @@ import java.util.List; @AllArgsConstructor @NoArgsConstructor @Document(indexName = "databaseindex", createIndex = false) -@Where(clause = "deleted is null") @EntityListeners(AuditingEntityListener.class) -@SQLDelete(sql = "update mdb_databases set deleted = NOW() where id = ?") @javax.persistence.Table(name = "mdb_databases", uniqueConstraints = { @UniqueConstraint(columnNames = {"id", "internalName"}) }) @@ -34,8 +32,8 @@ public class Database { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @ManyToOne(fetch = FetchType.EAGER, cascade = CascadeType.ALL) @@ -87,7 +85,4 @@ public class Database { @LastModifiedDate private Instant lastModified; - @Column - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/View.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/View.java index 0a60f175b545beb0ad48089db8667657a2ec6c78..ab289dd9d298f2b5b7b409c930ceae5dc364c2d5 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/View.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/View.java @@ -23,8 +23,8 @@ public class View { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Id @@ -69,7 +69,4 @@ public class View { @LastModifiedDate private Instant lastModified; - @Column - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/Table.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/Table.java index 77e548c47d350a8afdcb368795e5747cbf934b3b..eff1bce677e9300091244fc58c615d00dc83e760 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/Table.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/Table.java @@ -31,8 +31,8 @@ public class Table { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Id diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/TableColumn.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/TableColumn.java index eb13193b158f3a8b4b8b770b88a88d98dda24946..f74fb634a550883243c937523477a6272a1a067c 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/TableColumn.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/TableColumn.java @@ -32,8 +32,8 @@ public class TableColumn implements Comparable<TableColumn> { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Id diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/ColumnConcept.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/ColumnConcept.java index a2ed5eb24a7c6e246f49204d6039d4d8dca857ed..1afdedcd0fb775ac890bfb3bc16a1df83c82d204 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/ColumnConcept.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/ColumnConcept.java @@ -24,8 +24,8 @@ public class ColumnConcept implements Serializable { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long cid; @Id diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/Concept.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/Concept.java index 6def65fe5585c7b14062d1a2315378208238c311..00c984934c061b4c8ad89e5d677bba0578136dfd 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/Concept.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/database/table/columns/concepts/Concept.java @@ -23,8 +23,8 @@ public class Concept { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Column(name = "URI", nullable = false, columnDefinition = "TEXT") diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Creator.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Creator.java index 1681f68d4a2ecbc59af8be43b61732de44cd50db..b1bd315be00afd5ef0f3b0c9869c2b0ebfdc591e 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Creator.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Creator.java @@ -23,8 +23,8 @@ public class Creator { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Id diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Identifier.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Identifier.java index 950e54da3af72cc35b2ee0266e930107cf683c75..44c3a096e49c2edb567e3df91608f85612a755e2 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Identifier.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/Identifier.java @@ -23,9 +23,7 @@ import java.util.List; @ToString @AllArgsConstructor @NoArgsConstructor -@Where(clause = "deleted is null") @EntityListeners(AuditingEntityListener.class) -@SQLDelete(sql = "update mdb_identifiers set deleted = NOW() where id = ?") @javax.persistence.Table(name = "mdb_identifiers", uniqueConstraints = { @UniqueConstraint(columnNames = {"qid", "cid", "dbid"}) }) @@ -33,8 +31,8 @@ public class Identifier { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Column(name = "cid", nullable = false) @@ -133,9 +131,6 @@ public class Identifier { @LastModifiedDate private Instant lastModified; - @Column - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/RelatedIdentifier.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/RelatedIdentifier.java index ad318c4a22b5e80af96ce030f11ab16dba3a38ac..280715c956681a03fb03f791a4677e7c132b7f87 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/RelatedIdentifier.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/identifier/RelatedIdentifier.java @@ -19,7 +19,6 @@ import java.time.Instant; @AllArgsConstructor @NoArgsConstructor @IdClass(RelatedIdentifierKey.class) -@Where(clause = "deleted is null") @EqualsAndHashCode(onlyExplicitlyIncluded = true) @EntityListeners(AuditingEntityListener.class) @Table(name = "mdb_related_identifiers", uniqueConstraints = { @@ -29,8 +28,8 @@ public class RelatedIdentifier { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Id @@ -62,9 +61,6 @@ public class RelatedIdentifier { @LastModifiedDate private Instant lastModified; - @Column - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/TimeSecret.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/TimeSecret.java index c6922b85412353903cb9726150f5383fcdacdf68..53d71c8cc77d3412b531b7d0411aac1c080cdec0 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/TimeSecret.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/TimeSecret.java @@ -23,8 +23,8 @@ public class TimeSecret { @Id @EqualsAndHashCode.Include - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Column(nullable = false) diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/Token.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/Token.java index bacb80fc34b96f0f0bf3ac1740531d389f1e3a1c..241f16e7773af49616b4c0f3f628f9043dcf5af5 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/Token.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/Token.java @@ -16,7 +16,6 @@ import java.time.Instant; @AllArgsConstructor @NoArgsConstructor @ToString -@SQLDelete(sql = "update mdb_tokens set deleted = NOW() where id = ?") @EntityListeners(AuditingEntityListener.class) @EqualsAndHashCode(onlyExplicitlyIncluded = true) @Table(name = "mdb_tokens") @@ -24,9 +23,8 @@ public class Token { @Id @EqualsAndHashCode.Include - @Column(name = "id") - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(updatable = false, nullable = false) private Long id; @Column(nullable = false, updatable = false) @@ -50,7 +48,4 @@ public class Token { @Column(nullable = false, updatable = false) private Instant lastUsed; - @Column(nullable = false, updatable = false) - private Instant deleted; - } diff --git a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/User.java b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/User.java index 691bc2af6d7a61b982601aaa01c767cb7aae41c9..eb1f77d74acd36963aed939b09c29bf4260b2800 100644 --- a/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/User.java +++ b/fda-metadata-db/entities/src/main/java/at/tuwien/entities/user/User.java @@ -25,9 +25,8 @@ public class User { @Id @EqualsAndHashCode.Include - @Column(name = "userid") - @GenericGenerator(name = "native", strategy = "native") - @GeneratedValue(strategy = GenerationType.AUTO, generator = "native") + @GeneratedValue(strategy = GenerationType.IDENTITY) + @Column(name = "userid", updatable = false, nullable = false) private Long id; @Column(unique = true, nullable = false) diff --git a/fda-metadata-db/querystore/src/main/java/at/tuwien/querystore/View.java b/fda-metadata-db/querystore/src/main/java/at/tuwien/querystore/View.java index 6a45ec9d0e3af8761b832ebb0a4214ee57da0579..79419fca6e5a0ba5d667070501791ce21f56a36f 100644 --- a/fda-metadata-db/querystore/src/main/java/at/tuwien/querystore/View.java +++ b/fda-metadata-db/querystore/src/main/java/at/tuwien/querystore/View.java @@ -55,9 +55,6 @@ public class View implements Serializable { @Column(nullable = false) private String query; - @Column - private Instant deleted; - @Column(nullable = false, updatable = false) @CreatedDate private Instant created; diff --git a/fda-metadata-db/setup-schema.sql b/fda-metadata-db/setup-schema.sql index b8e6d24fe509b02674962900e82c0055cca6ced9..944a6b9e07a12e20aaca966e755df67182ae66bc 100644 --- a/fda-metadata-db/setup-schema.sql +++ b/fda-metadata-db/setup-schema.sql @@ -1,141 +1,8 @@ BEGIN; -CREATE SEQUENCE mdb_images_environment_item_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_images_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_images_date_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_containers_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_user_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_user_role_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_data_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_databases_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_tables_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_columns_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_columns_enum_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_view_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_columns_concepts_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_identifiers_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_related_identifiers_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_creators_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_time_secrets_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_tokens_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - -CREATE SEQUENCE mdb_concepts_seq - START WITH 1 - INCREMENT BY 1 - NO MINVALUE - NO MAXVALUE - CACHE 1; - CREATE TABLE IF NOT EXISTS mdb_users ( - UserID bigint not null DEFAULT nextval(mdb_user_seq), + UserID bigint not null AUTO_INCREMENT, external_id VARCHAR(255) UNIQUE, OID bigint, username VARCHAR(255) not null, @@ -160,7 +27,7 @@ CREATE TABLE IF NOT EXISTS mdb_users CREATE TABLE mdb_images ( - id bigint NOT NULL DEFAULT nextval(mdb_images_seq), + id bigint NOT NULL AUTO_INCREMENT, repository character varying(255) NOT NULL, tag character varying(255) NOT NULL, default_port integer NOT NULL, @@ -178,7 +45,7 @@ CREATE TABLE mdb_images CREATE TABLE mdb_time_secrets ( - id bigint not null default nextval(mdb_time_secrets_seq), + id bigint not null AUTO_INCREMENT, uid bigint not null, token character varying(255) NOT NULL, processed boolean NOT NULL default false, @@ -190,20 +57,19 @@ CREATE TABLE mdb_time_secrets CREATE TABLE mdb_tokens ( - id bigint not null default nextval(mdb_tokens_seq), + id bigint not null AUTO_INCREMENT, token_hash varchar(255) NOT NULL, creator bigint not null, created timestamp NOT NULL DEFAULT NOW(), expires timestamp NOT NULL, last_used timestamp, - deleted timestamp, PRIMARY KEY (id), FOREIGN KEY (creator) REFERENCES mdb_users (UserID) ); CREATE TABLE mdb_images_date ( - id bigint NOT NULL DEFAULT nextval(mdb_images_date_seq), + id bigint NOT NULL AUTO_INCREMENT, iid bigint NOT NULL, database_format character varying(255) NOT NULL, unix_format character varying(255) NOT NULL, @@ -217,7 +83,7 @@ CREATE TABLE mdb_images_date CREATE TABLE IF NOT EXISTS mdb_containers ( - id bigint NOT NULL DEFAULT nextval(mdb_containers_seq), + id bigint NOT NULL AUTO_INCREMENT, HASH character varying(255) NOT NULL, INTERNAL_NAME character varying(255) NOT NULL, NAME character varying(255) NOT NULL, @@ -227,7 +93,6 @@ CREATE TABLE IF NOT EXISTS mdb_containers created timestamp NOT NULL DEFAULT NOW(), created_by bigint NOT NULL, LAST_MODIFIED timestamp, - deleted timestamp, PRIMARY KEY (id), FOREIGN KEY (created_by) REFERENCES mdb_users (UserID), FOREIGN KEY (image_id) REFERENCES mdb_images (id) @@ -235,7 +100,7 @@ CREATE TABLE IF NOT EXISTS mdb_containers CREATE TABLE mdb_images_environment_item ( - id bigint NOT NULL DEFAULT nextval(mdb_images_environment_item_seq), + id bigint NOT NULL AUTO_INCREMENT, `key` character varying(255) NOT NULL, value character varying(255) NOT NULL, etype ENUM ('PRIVILEGED_USERNAME', 'PRIVILEGED_PASSWORD', 'USERNAME', 'PASSWORD') NOT NULL, @@ -248,7 +113,7 @@ CREATE TABLE mdb_images_environment_item CREATE TABLE IF NOT EXISTS mdb_data ( - ID bigint DEFAULT nextval(mdb_data_seq), + ID bigint NOT NULL AUTO_INCREMENT, PROVENANCE TEXT, FileEncoding TEXT, FileType VARCHAR(100), @@ -278,7 +143,7 @@ CREATE TABLE IF NOT EXISTS mdb_licenses CREATE TABLE IF NOT EXISTS mdb_databases ( - id bigint NOT NULL DEFAULT nextval(mdb_databases_seq), + id bigint NOT NULL AUTO_INCREMENT, name character varying(255) NOT NULL, internal_name character varying(255) NOT NULL, exchange character varying(255) NOT NULL, @@ -289,7 +154,6 @@ CREATE TABLE IF NOT EXISTS mdb_databases Contactperson BIGINT, created timestamp NOT NULL DEFAULT NOW(), last_modified timestamp, - deleted timestamp NULL, PRIMARY KEY (id), FOREIGN KEY (Creator) REFERENCES mdb_users (UserID), FOREIGN KEY (Contactperson) REFERENCES mdb_users (UserID), @@ -305,7 +169,7 @@ CREATE TABLE IF NOT EXISTS mdb_databases_subjects CREATE TABLE IF NOT EXISTS mdb_tables ( - ID bigint NOT NULL DEFAULT nextval(mdb_tables_seq), + ID bigint NOT NULL AUTO_INCREMENT, tDBID bigint NOT NULL, internal_name character varying(255) NOT NULL, topic character varying(255) NOT NULL, @@ -323,14 +187,14 @@ CREATE TABLE IF NOT EXISTS mdb_tables created timestamp NOT NULL DEFAULT NOW(), created_by bigint NOT NULL, last_modified timestamp, - PRIMARY KEY (tDBID, ID), + PRIMARY KEY (ID, tDBID), FOREIGN KEY (created_by) REFERENCES mdb_users (UserID), FOREIGN KEY (tDBID) REFERENCES mdb_databases (id) ); CREATE TABLE IF NOT EXISTS mdb_columns ( - ID bigint NOT NULL DEFAULT nextval(mdb_columns_seq), + ID bigint NOT NULL AUTO_INCREMENT, cDBID bigint NOT NULL, tID bigint NOT NULL, dfID bigint, @@ -350,12 +214,12 @@ CREATE TABLE IF NOT EXISTS mdb_columns last_modified timestamp, FOREIGN KEY (cDBID, tID) REFERENCES mdb_tables (tDBID, ID), FOREIGN KEY (created_by) REFERENCES mdb_users (UserID), - PRIMARY KEY (cDBID, tID, ID) + PRIMARY KEY (ID, cDBID, tID) ); CREATE TABLE IF NOT EXISTS mdb_columns_enums ( - ID bigint NOT NULL DEFAULT nextval(mdb_columns_enum_seq), + ID bigint NOT NULL AUTO_INCREMENT, eDBID bigint NOT NULL, tID bigint NOT NULL, cID bigint NOT NULL, @@ -411,7 +275,7 @@ CREATE TABLE IF NOT EXISTS mdb_columns_cat CREATE TABLE IF NOT EXISTS mdb_concepts ( - id bigint not null default nextval(mdb_concepts_seq), + id bigint not null AUTO_INCREMENT, URI TEXT, name VARCHAR(255), created timestamp NOT NULL DEFAULT NOW(), @@ -435,7 +299,7 @@ CREATE TABLE IF NOT EXISTS mdb_columns_concepts CREATE TABLE IF NOT EXISTS mdb_view ( - id bigint NOT NULL DEFAULT nextval(mdb_view_seq), + id bigint NOT NULL AUTO_INCREMENT, vcid bigint NOT NULL, vdbid bigint NOT NULL, vName VARCHAR(255) NOT NULL, @@ -447,16 +311,15 @@ CREATE TABLE IF NOT EXISTS mdb_view InitialView BOOLEAN NOT NULL, created timestamp NOT NULL DEFAULT NOW(), last_modified timestamp, - deleted timestamp, created_by bigint NOT NULL, FOREIGN KEY (created_by) REFERENCES mdb_users (UserID), FOREIGN KEY (vdbid) REFERENCES mdb_databases (id), - PRIMARY KEY (vdbid, id) + PRIMARY KEY (id, vdbid) ); CREATE TABLE IF NOT EXISTS mdb_identifiers ( - id bigint DEFAULT nextval(mdb_identifiers_seq), + id bigint NOT NULL AUTO_INCREMENT, cid bigint NOT NULL, dbid bigint NOT NULL, qid bigint, @@ -480,7 +343,6 @@ CREATE TABLE IF NOT EXISTS mdb_identifiers created timestamp NOT NULL DEFAULT NOW(), created_by bigint NOT NULL, last_modified timestamp, - deleted timestamp, PRIMARY KEY (id), /* must be a single id from persistent identifier concept */ FOREIGN KEY (cid) REFERENCES mdb_containers (id), FOREIGN KEY (dbid) REFERENCES mdb_databases (id), @@ -490,7 +352,7 @@ CREATE TABLE IF NOT EXISTS mdb_identifiers CREATE TABLE IF NOT EXISTS mdb_related_identifiers ( - id bigint DEFAULT nextval(mdb_related_identifiers_seq), + id bigint NOT NULL AUTO_INCREMENT, iid bigint NOT NULL, value varchar(255) NOT NULL, type varchar(255), @@ -498,7 +360,6 @@ CREATE TABLE IF NOT EXISTS mdb_related_identifiers created timestamp NOT NULL DEFAULT NOW(), created_by bigint NOT NULL, last_modified timestamp, - deleted timestamp, PRIMARY KEY (id, iid), /* must be a single id from persistent identifier concept */ FOREIGN KEY (iid) REFERENCES mdb_identifiers (id), FOREIGN KEY (created_by) REFERENCES mdb_users (UserID) @@ -506,7 +367,7 @@ CREATE TABLE IF NOT EXISTS mdb_related_identifiers CREATE TABLE IF NOT EXISTS mdb_creators ( - id bigint DEFAULT nextval(mdb_creators_seq), + id bigint NOT NULL AUTO_INCREMENT, pid bigint NOT NULL, name VARCHAR(255) NOT NULL, affiliation VARCHAR(255), diff --git a/fda-metadata-db/setup-user.sh b/fda-metadata-db/setup-user.sh deleted file mode 100644 index 724f28f273cbf917a886566a95241d462624153b..0000000000000000000000000000000000000000 --- a/fda-metadata-db/setup-user.sh +++ /dev/null @@ -1,6 +0,0 @@ -#!/bin/sh -mysql --user="root" --password="${METADATA_PASSWORD}" --database="${METADATA_DB}" << EOF -CREATE USER '${MARIADB_USERNAME}' IDENTIFIED BY '${MARIADB_PASSWORD}'; -GRANT SELECT ON ${METADATA_DB}.* TO '${MARIADB_USERNAME}'; -FLUSH PRIVILEGES; -EOF \ No newline at end of file