Customize Client Identifier via LocalStorage

[dmoruzzi]

Describe your changes

This introduces the ability to override the default clientId in the sforce-call-options header. Before this change, the default value is always client=Salesforce Inspector Reloaded. After this change, users can insert a key called "ClientId" in the browser's LocalStorage to dynamically set any value for the client. Empty value with key will be treated as null by Salesforce.

1. This enhances the development of this extension by allowing me to easily self-document in the API call itself.
2. Testing teams can set ClientId easily to enhance traceability of queries in the ApiEvent records.

Issue ticket number and link

Feature 504

Checklist before requesting a review

• I have read and understand the Contributions section
• Target branch is releaseCandidate and not master
• I have performed a self-review of my code
• I ran the unit tests and my PR does not break any tests
• I documented the changes I've made on the CHANGES.md and followed actual conventions
• I added a new section on how-to.md (optional)

[tprouvot]

Hi @dmoruzzi,
I understand that you want to define your own clientId with this PR. What I don't really get is why this should be merged in the extension ?

Keeping the local update without merging them is not convenient ?
Are you working on some features that requires a particular analysis of the api calls ?

I would prefer end users not able to update this clientId, and the fact that you did not exposed this in the options is a good thing, I would also remove all documentation (CHANGES.md, screenshot, how-to) about this and let only advanced users customize the value.

[dmoruzzi]

Are you working on some features that requires a particular analysis of

the api calls ? Not as a part of this extension. My use case is we send all of the ApiEvent records to datadog. Occasionally we need to audit the testing results, and having a unique ClientId per testing situation would easily allow these adhoc runs to be filtered. I was thinking about adding a page to download the most recent ApiTotalUsage report and getting a catalog of users/client/usage. But the CSV format can become too large for production organizations that we pivoted. We ultimately just put that data into a Confluence webpage instead. In regards to the documentation, we can remove it and only expose to advance users. This would still be an enhancement compared to our team modifying the JS file and then reloading the extension. Would you prefer we closed this instead? Alternatively we could do something like simple_salesforce and add an option and prefix with "Salesforce Inspector Reloaded/{variableClientId}"? This keeps the use case of identifying API calls while still allowing quick filtering.

…

On Mon, Jun 2, 2025, 4:17 AM Thomas Prouvot ***@***.***> wrote: *tprouvot* left a comment (tprouvot/Salesforce-Inspector-reloaded#866) <#866 (comment)> Hi @dmoruzzi <https://github.com/dmoruzzi>, I understand that you want to define your own clientId with this PR. What I don't really get is why this should be merged in the extension ? Keeping the local update without merging them is not convenient ? Are you working on some features that requires a particular analysis of the api calls ? I would prefer end users not able to update this clientId, and the fact that you did not exposed this in the options is a good thing, I would also remove all documentation (CHANGES.md, screenshot, how-to) about this and let only advanced users customize the value. — Reply to this email directly, view it on GitHub <#866 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFK6CXYGMUQW7YMTXQJD3BL3BQCABAVCNFSM6AAAAAB6KRQY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSMRZGM3TENBVGQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[tprouvot]

I'm OK with keeping this one opened, you just need to remove the documentation.
I like the idea of keeping SIR as clientId prefix ("Salesforce Inspector Reloaded/{variableClientId}") 👍

[tprouvot]

Hi @dmoruzzi
Did you had a chance to read my last comment ?

[dmoruzzi]

Hello. I have read this message and I agree with it. I just haven't found the time, I've been swamped with work. Thank you :)

…

On Tue, Jun 24, 2025, 12:28 PM Thomas Prouvot ***@***.***> wrote: *tprouvot* left a comment (tprouvot/Salesforce-Inspector-reloaded#866) <#866 (comment)> Hi @dmoruzzi <https://github.com/dmoruzzi> Did you had a chance to read my last comment ? — Reply to this email directly, view it on GitHub <#866 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFK6CX5XLZXAUFAFE3J6EXL3FF4EBAVCNFSM6AAAAAB6KRQY4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTAMBRGE2DAMZVGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>