EmbeddedChat icon indicating copy to clipboard operation
EmbeddedChat copied to clipboard

Issue: Add Cache Breaking Notice for Removed Query Parameter in API

Open divysaxena24 opened this issue 3 months ago • 4 comments

Location

File: apps/meteor/app/api/server/v1/im.ts
Line: 537


Problem

A TODO comment indicates that a cache-breaking notice should be added because a query parameter was removed from the API.
Currently, this missing notice can cause confusion for developers upgrading or using cached API responses.

Existing Code:

// TODO: CACHE: Add Breaking notice since we removed the query param

// Proposed Solution
// Add appropriate deprecation notice
// NOTICE: This endpoint parameter was removed due to security concerns and 
// performance optimization.
// The feature was replaced with a more secure implementation in subsequent versions.
// Please refer to the API documentation for migration details.

divysaxena24 avatar Nov 04 '25 18:11 divysaxena24

Hi @divysaxena24 , I’d like to work on this issue. Could you please assign it to me?

AnirbanSinha27 avatar Nov 09 '25 15:11 AnirbanSinha27

Can you share, where this warning is appearing ?

Spiral-Memory avatar Nov 27 '25 15:11 Spiral-Memory

@Spiral-Memory its in the main RocketChat repo

AnirbanSinha27 avatar Nov 28 '25 15:11 AnirbanSinha27

Then why is it being shared here 😅

Spiral-Memory avatar Nov 28 '25 16:11 Spiral-Memory