Skip to content

Add auth in Rewriter client#10

Merged
silvadenisaraujo merged 1 commit intomasterfrom
bugfix/add-auth-rewriter-client
Sep 25, 2025
Merged

Add auth in Rewriter client#10
silvadenisaraujo merged 1 commit intomasterfrom
bugfix/add-auth-rewriter-client

Conversation

@silvadenisaraujo
Copy link
Contributor

@silvadenisaraujo silvadenisaraujo commented Sep 23, 2025

What is the purpose of this pull request?

Adds authentication token in the request for Rewriter mutations.

What problem is this solving?

Since Rewriter added authentication in its mutation the plugin started to break due to missing authentication token.

How should this be manually tested?

Follow the readme to build the plugin and link it using vtex plugins link
Using a csv file as an example run: vtex redirects import redirects.csv
Then delete the redirects uploaded: vtex redirects delete redirects.csv

Screenshots or example usage

image image

Types of changes

  • Refactor (non-breaking change that only makes the code better)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

Chores checklist

  • Update CHANGELOG.md

@silvadenisaraujo silvadenisaraujo force-pushed the bugfix/add-auth-rewriter-client branch from 74072ef to cd2b8cb Compare September 23, 2025 20:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds authentication support to the Rewriter client by including the VtexIdclientAutCookie header in requests. This fixes a breaking issue where the plugin stopped working after the Rewriter service added authentication requirements to its mutation operations.

Key changes:

  • Adds authentication token to Rewriter client headers
  • Updates changelog to document the fix

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/clients/apps/Rewriter.ts Adds VtexIdclientAutCookie header with auth token to fix authentication issues
CHANGELOG.md Documents the authentication header addition in the Fixed section

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@silvadenisaraujo silvadenisaraujo marked this pull request as ready for review September 23, 2025 20:42
Copy link

@arturpimentel arturpimentel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gogogogogo

@silvadenisaraujo silvadenisaraujo merged commit 6a36286 into master Sep 25, 2025
5 checks passed
@silvadenisaraujo silvadenisaraujo deleted the bugfix/add-auth-rewriter-client branch September 25, 2025 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants