Skip to content

Commit

Permalink
feat: Deprecate --use-artifact-bundle option
Browse files Browse the repository at this point in the history
  • Loading branch information
szokeasaurusrex committed Jan 20, 2025
1 parent b26e0f2 commit a907e47
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/commands/sourcemaps/upload.rs
Original file line number Diff line number Diff line change
Expand Up @@ -224,14 +224,14 @@ pub fn make_command(command: Command) -> Command {
uploaded.",
),
)
// NOTE: Hidden until we decide to expose it publicly
.arg(
Arg::new("use_artifact_bundle")
.long("use-artifact-bundle")
.action(ArgAction::SetTrue)
.help(
"Use new Artifact Bundles upload, that enables the use of Debug IDs \
for Source Maps discovery.",
"[DEPRECATED] Force artifact bundles to be used for upload, even when not \
supported by the server. This option has always only been intended for \
internal use, and it is now officially deprecated.",
)
.hide(true),
)
Expand Down Expand Up @@ -429,6 +429,9 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
if matches.get_flag("use_artifact_bundle")
|| env::var("SENTRY_FORCE_ARTIFACT_BUNDLES").ok().as_deref() == Some("1")
{
log::warn!("The --use-artifact-bundle option and the SENTRY_FORCE_ARTIFACT_BUNDLES environment variable \
are both deprecated, and both will be removed in the next major version.");

if let Some(ref mut options) = chunk_upload_options {
if !options.supports(ChunkUploadCapability::ArtifactBundles) {
options.accept.push(ChunkUploadCapability::ArtifactBundles);
Expand Down

0 comments on commit a907e47

Please sign in to comment.