Quite confusing log message

[lforst]

WARN 2024-09-02 14:41:35.566111677 +00:00 Using https://sentry.io (embedded in token) rather than manually-configured URL https://sentry.io. To use https://sentry.io, please provide an auth token for this URL.

I am getting this in our Changelog Vercel logs. https://vercel.com/sentry/changelog/DT2tEb1huCLLTUm7DzDaWawhZHzi#L159

IMO, if the manually provided URL, and the URL in the auth token are matching, we should not print this message.

cc @szokeasaurusrex

[szokeasaurusrex]

strange – I thought I was only printing this when the two URLs are different, but I will double check

[szokeasaurusrex]

Okay, looks like in one of the places we emit this message we check whether the URLs are different, but in the other spot, we don't:

sentry-cli/src/config.rs

Lines 74 to 83 in 288382f

	let url = if token_url.is_empty() {
	default_url
	} else {
	if !default_url.is_empty() {
	log::warn!(
	"Using {token_url} (embedded in token) rather than manually-configured URL {default_url}. \
	To use {default_url}, please provide an auth token for this URL."
	);
	}
	token_url.into()

Should be a pretty simple fix to add this check, thanks for raising @lforst!

[dmadridy]

Hey, just got this message as well when configuring Sentry. Let me know when its solved. Thanks!

[szokeasaurusrex]

@dmadridy hopefully we will get a fix out this week!