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

Ticket: 23849 #72

Closed
wants to merge 6 commits into from
Closed

Ticket: 23849 #72

wants to merge 6 commits into from

Conversation

Nesma1988
Copy link
Contributor

Fixes: https://bugs.freenas.org/issues/23849
Description: Add alerts to Support Ticket tab for files that are "too large"

NOTE:
In order to fetch the size of the debug file we will need the back-end team to add this header Access-Control-Expose-Headers: ETag

You can refer this stack overflow thread as well.
https://stackoverflow.com/questions/5822985/cross-domain-resource-sharing-get-refused-to-get-unsafe-header-etag-from-re

@kmoore134
Copy link
Contributor

Nesma, where is that option enabled? In the angular side or web server?

@Nesma1988
Copy link
Contributor Author

@kmoore134

On the server side and Access-Control-Expose-Headers: ETag this header must be added in the response headers.

@vaibhav-rbs
Copy link
Contributor

as part of submitting ticket thru support portal, we have users wait as shown in screenshot, please reimplement that thanks see corresponding ticket for details.

@vaibhav-rbs
Copy link
Contributor

screen shot 2017-11-09 at 11 14 12 am

Copy link
Contributor

@vaibhav-rbs vaibhav-rbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as part of submitting ticket thru support portal, we have users wait as shown in screenshot, please reimplement that thanks see corresponding ticket for details.

@Nesma1988 Nesma1988 closed this Nov 29, 2017
@kmoore134 kmoore134 deleted the enhancements_file-size-check branch August 22, 2018 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants