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

Release v0.2.0 #240

Merged
merged 1 commit into from
Sep 5, 2022
Merged

Release v0.2.0 #240

merged 1 commit into from
Sep 5, 2022

Conversation

SuperQ
Copy link
Collaborator

@SuperQ SuperQ commented Sep 5, 2022

Release [v0.2.0] - 2022-09-05
** Bugfix

  • fixed for PR#238
  • fixed for PR#238
  • fixed issues/204 that syntax error has occured
  • rollback to 549cc4d
  • fixed issues/137, issues/98 that maxSize in cacheZones is displayed incorrectly
  • fixed issues/174 that XSS vulnerability in the html page Feature: added moduleVersion field in format/json
  • added escape strings for filter names in JSON
  • fixed the sum value of histogram in upstream metrics
  • fixed to display all A records of server without zone directive in the upstream block.

** Chore

  • Use git-chglog

** Comment

  • added moduleVersion
  • added additional information about cacheZones
  • added tested versions
  • added a diagram for the order of module directives

** Compatibility

  • fixed ngx_http_vhost_traffic_status_display_get_upstream_nelts() to calculate all A records of server.

** Docs

  • Fix README

** Docs

  • fix simple typo, destory -> destroy

** Fix

  • limit the r->uri search scope to avoid overflow

** Prometheus

  • fix nginx_vts_filter_requests_total labels
  • remove request "total" metrics

** Refactor

  • changed version
  • changed spacing
  • changed spacing
  • changed if statement from merged pull/145

** Test

  • describe how to test and fix failed test case

Signed-off-by: SuperQ superq@gmail.com

@u5surf
Copy link
Collaborator

u5surf commented Sep 5, 2022

@SuperQ
In the README, I've found the version number hard written. What should we do that?

## Version
This document describes nginx-module-vts `v0.1.18` released on 22 Jun 2018.

@SuperQ
Copy link
Collaborator Author

SuperQ commented Sep 5, 2022

I'll make a link to the Releases page.

Release [v0.2.0] - 2022-09-05
** Bugfix
- fixed for PR[#238](#238)
- fixed for PR[#238](#238)
- fixed issues/204 that syntax error has occured
- rollback to 549cc4d
- fixed issues/137, issues/98 that maxSize in cacheZones is displayed incorrectly
- fixed issues/174 that XSS vulnerability in the html page Feature: added moduleVersion field in format/json
- added escape strings for filter names in JSON
- fixed the sum value of histogram in upstream metrics
-  fixed to display all A records of server without zone directive in the upstream block.

** Chore
- Use git-chglog

** Comment
- added moduleVersion
- added additional information about cacheZones
- added tested versions
- added a diagram for the order of module directives

** Compatibility
- fixed ngx_http_vhost_traffic_status_display_get_upstream_nelts() to calculate all A records of server.

** Docs
- Fix README

** Docs
- fix simple typo, destory -> destroy

** Fix
- limit the r->uri search scope to avoid overflow

** Prometheus
- fix nginx_vts_filter_requests_total labels
- remove request "total" metrics

** Refactor
- changed version
- changed spacing
- changed spacing
- changed if statement from merged pull/145

** Test
- describe how to test and fix failed test case

Signed-off-by: SuperQ <superq@gmail.com>
@u5surf
Copy link
Collaborator

u5surf commented Sep 5, 2022

@SuperQ LGTM, how about @vozlt ?

Copy link
Owner

@vozlt vozlt left a comment

Choose a reason for hiding this comment

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

Thanks for your work.

@SuperQ SuperQ requested a review from u5surf September 5, 2022 13:20
@u5surf u5surf merged commit 6dadfc8 into master Sep 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants