This message was deleted.
# community-supports
Slackbot
05/03/2023, 9:27 AMThis message was deleted.
a
Adam
05/03/2023, 9:45 AMis the
SENTRY_AUTH_TOKENbuild.gradle(.kts)d
Daniele Segato
05/03/2023, 9:48 AMNo, it is used by some Gradle plugin i use and I've no control of
Daniele Segato
05/03/2023, 9:49 AMThe "inject environment variable through file" seems quite a common use case for a build system, I'm surprised this wasn't well documented
Daniele Segato
05/03/2023, 9:50 AMBtw the plugin add a task to my build, that task go look for the environment variable
a
Adam
05/03/2023, 9:52 AMdoes the Gradle Plugin expose a property where you can manually set the Sentry auth token value, or does it always fetch it from the environment variable?
d
Daniele Segato
05/03/2023, 9:53 AMIt look for a properties file inside the my module. But I already need that to set other properties that are committed.
Daniele Segato
05/03/2023, 9:54 AMAre you saying there is no way of loading environment variables in Gradle via file?
a
Adam
05/03/2023, 9:55 AMif it’s possible to set it as a property, e.g.
Copy code
// build.gradle.kts
sentryExtension {
authToken.set("...")
} Copy code
// build.gradle.kts
sentryExtension {
authToken.set(providers.gradleProperty("authToken"))
}ORG_GRADLE_PROJECT_authToken=123authToken=123$GRADLE_USER_HOME/gradle.propertiesd
Daniele Segato
05/03/2023, 9:57 AMNo it's not.
The task internally execute a command line tool and that command line tool will look for the environment variable
a
Adam
05/03/2023, 9:59 AMif you have to use an environment variable then you could try to create a init script that would set the environment variable, but I don’t think that’s easy in Java https://stackoverflow.com/questions/318239/how-do-i-set-environment-variables-from-java
Adam
05/03/2023, 10:00 AMThe task internally execute a command line tool and that command line tool will look for the environment variableah well, it is possible to set the environment variables for launched processes
Adam
05/03/2023, 10:01 AM Copy code
val myExecTask by tasks.registering(Exec::class) {
environment("SENTRY_AUTH_TOKEN", providers.gradleProperty("authToken").get())
}d
Daniele Segato
05/03/2023, 10:10 AMThanks I've entered a work call but I'll look into it
👍 1
Daniele Segato
05/03/2023, 10:11 AMWhere can I make feature requests to Gradle?
v
Vampire
05/03/2023, 10:12 AMVampire
05/03/2023, 10:13 AMEnvironment variables are to be supplied by the environment, like the CI server. That's why a running Java process cannot modify the environment variables, unless it forks a new process. Actually Gradle forks a new process from the CLI, the daemon where it then probably could set environment variables.
Vampire
05/03/2023, 10:13 AMBut you can only configure the JVM arguments that are used for that, because environment variables are, well, meant to be supplied by the environment.
Vampire
05/03/2023, 10:14 AMA better place for a feature request would be the tool that requires it as environment variable, or the plugin that calls that tool, so that either of those supports a different way to supply that value.
a
Adam
05/03/2023, 10:16 AMit looks like Sentry CLI supports both an environment variable and a config file? https://github.com/getsentry/sentry-docs/blob/master/src/docs/product/cli/configuration.mdx#to-authenticate-manually
v
Vampire
05/03/2023, 10:27 AMYou could change the
gradlewd
Daniele Segato
05/03/2023, 11:06 AMThe config file I already use for stuff that has to be committed.
Daniele Segato
05/03/2023, 11:06 AMI think this should be supported by Gradle or at least Gradle wrapper, it's such a common need
v
Vampire
05/03/2023, 11:07 AMAs I described, it is an absolutely uncommon need.
Vampire
05/03/2023, 11:08 AMEnvironment variables are to be supplied by the environment
Vampire
05/03/2023, 11:08 AMBut feel free to open a feature request, that's just my personal opinion
d
Daniele Segato
05/03/2023, 11:08 AMFor instance, the gradlew could automatically look for a .env file and load it or, even better, multiple .env files (there are other tools that do that and already have a best practice for it)
v
Vampire
05/03/2023, 11:12 AMAs I said, feel free to open a feature request and see what the Gradle guys say. But in Gradle you already have file-based environment-specific key-value pairs, so, ... 🙂
d
Daniele Segato
05/03/2023, 2:51 PMMy feature request https://github.com/gradle/gradle/issues/24976
👌 1
r
Roman Zavarnitsyn
05/04/2023, 6:52 PMhi @Daniele Segato, Roman from Sentry here 👋 we already have a feature request for that specific case, so I guess I'll just throw a quick PR for that soon, seeing people have a demand for that. https://github.com/getsentry/sentry-android-gradle-plugin/issues/440
❤️ 1
Roman Zavarnitsyn
05/04/2023, 6:52 PMbut we won't advertise it much I guess, because there's a risk of leaking tokens if someone sets them directly in the extension lol
d
Daniele Segato
05/04/2023, 6:56 PMThanks @Roman Zavarnitsyn I subscribed to the issue and I'll switch to that when you release it. I still think from the Gradle standpoint it makes sense to support setting env variables via file.
What do you mean about the leaking?
r
Roman Zavarnitsyn
05/04/2023, 6:58 PMif you set
sentry.authToken="<my_real_token>"build.gradled
Daniele Segato
05/04/2023, 6:58 PMOh yeah, well that can happen with the
sentry.propertiesr
Roman Zavarnitsyn
05/04/2023, 7:08 PMtrue!
7 Views