Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Concerns over SaveMethod technique in ASP.NET Core #530

Open
MaximBalaganskiy opened this issue Dec 15, 2022 · 2 comments
Open

Concerns over SaveMethod technique in ASP.NET Core #530

MaximBalaganskiy opened this issue Dec 15, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@MaximBalaganskiy
Copy link

I'm a bit concerned regarding the usage of SaveMethod callback.
In a real-life scenario this callback will capture a controller reference.
If the controller allocated any resources, they won't be cleaned up until the WebReport is expelled from the cache.
Somehow, an idea that an old controller instance will be hanging around makes me uneasy.
Are there any alternatives to this report saving method?
I can modify the online designer code slightly to pass the report template to the iframe parent and handle saving from the SPA itself,
but would prefer an out of the box solution.

@MaximBalaganskiy MaximBalaganskiy added the enhancement New feature or request label Dec 15, 2022
@Alekarfes
Copy link

Thank you for your suggestions, we will take a look at them.

@MaximBalaganskiy
Copy link
Author

Have you found the time to look into this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants