`FileCacheOptions` not working after the header was introduced in FileCache settings in version 23.0.0 about ocelot

[ggnaegi]

@raman-m it's probably related to the fact that the body hashing is now disabled by default and it must be activated explicitly. But, as mentioned by @thiagoloureiro, the needed property isn't available in file configuration. So, I would recommend to deal with this issue before anything else...

[sberwal]

Hi Team

we have also observed the same behaviour. It seems the request payload is not considered when caching the data.

Hence cache returns the data for 1st request payload from the cache, for subsequent requests with different request payload.

This is happening when Header is not set!

Thanks
Suraj

[raman-m]

@SheruGaur

Please can you help us fix this as this seems to be a breaking change since version 23.0.0 as mentioned by Raman in the discussion.

Certainly...

1. Can you provide the specific lines of code where the breaking change occurs?
2. We are willing to assist you if you can contribute further by demonstrating the precise breaking change in a PR, could you do that?

[raman-m]

@ggnaegi commented on May 9

Understood! I guess we need to link the PR 2058 ... The bug fix will be released in current milestone. I don't want to make separate hotfix release: developers may to rollback to previous versions temporarily. So, the bugfix will be a part of version 23.3

P.S.

Ocelot/src/Ocelot/Configuration/File/FileRoute.cs

Line 45 in aef3e6b

public FileCacheOptions FileCacheOptions { get; set; }

Firstly added in commit 47afc85#diff-e1943063e63a6438191010c566111bea5c47f4cb063c04c7cc6c91a8c4267812 by Tom many many years ago.
I wonder the property was named with File prefix. And the most appropriate name is CacheOptions.
Will renaming be a breaking change?
It will be better to rename 👉

- public FileCacheOptions FileCacheOptions { get; set; }
+ public FileCacheOptions CacheOptions { get; set; }