EmbeddedChat
EmbeddedChat copied to clipboard
Issue: Add Cache Breaking Notice for Removed Query Parameter in API
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.
Hi @divysaxena24 , I’d like to work on this issue. Could you please assign it to me?
Can you share, where this warning is appearing ?
@Spiral-Memory its in the main RocketChat repo
Then why is it being shared here 😅