Skip to content

Commit

Permalink
Remove sentry integration (#3739)
Browse files Browse the repository at this point in the history
  • Loading branch information
carlesarnal authored Oct 17, 2023
1 parent 3f9b399 commit 3619a36
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 56 deletions.
6 changes: 0 additions & 6 deletions app/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -141,12 +141,6 @@
<artifactId>quarkus-agroal</artifactId>
</dependency>

<!-- Sentry -->
<dependency>
<groupId>io.sentry</groupId>
<artifactId>sentry</artifactId>
</dependency>

<!-- Third Party Libraries -->
<dependency>
<groupId>commons-codec</groupId>
Expand Down

This file was deleted.

4 changes: 0 additions & 4 deletions app/src/main/resources/application.properties
Original file line number Diff line number Diff line change
Expand Up @@ -110,9 +110,6 @@ quarkus.http.access-log.pattern="apicurio-registry.access method="%{METHOD}" pat
#this property will be used by Quarkus 2.X
quarkus.http.access-log.exclude-pattern=/health/.*

# Sentry - the rest of the sentry configuration is picked from sentry own env vars
registry.enable.sentry=${ENABLE_SENTRY:false}

## UI location override
registry.ui.root=/ui/
registry.ui.config.uiContextPath=${registry.ui.root}
Expand Down Expand Up @@ -189,5 +186,4 @@ app.authn.token.endpoint=${registry.auth.token.endpoint}
app.authn.client-secret=${registry.auth.client-secret}
app.authn.client-id=${quarkus.oidc.client-id}

app.enable.sentry=${registry.enable.sentry}
app.authn.basic-auth-client-credentials.enabled.dynamic.allow=${registry.config.dynamic.allow-all}
6 changes: 0 additions & 6 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,6 @@
<httpclient.version>4.5.14</httpclient.version>
<apicurio-common-rest-client.version>0.1.18.Final</apicurio-common-rest-client.version>
<apicurio-common-app-components.version>0.2.2.Final</apicurio-common-app-components.version>
<sentry.version>1.7.30</sentry.version>
<kafka-oauth-client.version>0.14.0</kafka-oauth-client.version>
<kafka-clients.version>3.5.1</kafka-clients.version>
<snakeyaml.version>2.1</snakeyaml.version>
Expand Down Expand Up @@ -623,11 +622,6 @@
<artifactId>httpclient</artifactId>
<version>${httpclient.version}</version>
</dependency>
<dependency>
<groupId>io.sentry</groupId>
<artifactId>sentry</artifactId>
<version>${sentry.version}</version><!--$NO-MVN-MAN-VER$-->
</dependency>
<dependency>
<groupId>org.eclipse.jgit</groupId>
<artifactId>org.eclipse.jgit</artifactId>
Expand Down
6 changes: 1 addition & 5 deletions ui/src/services/config/config.type.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,18 +56,14 @@ export type AlertProps = {
* Unique request ID.
*/
requestId?: string;
/**
* Unique sentry error ID.
*/
sentryId?: string;
/**
* data-testid attribute
*/
dataTestId?: string;
};

export declare type Alerts = {
addAlert: ({ id, title, variant, description, dataTestId, autoDismiss, dismissable, dismissDelay, requestId, sentryId }: AlertProps) => void;
addAlert: ({ id, title, variant, description, dataTestId, autoDismiss, dismissable, dismissDelay, requestId }: AlertProps) => void;
};


Expand Down

0 comments on commit 3619a36

Please sign in to comment.