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

tweak: allows custom names to pass tls verification #1562

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

mornyx
Copy link
Collaborator

@mornyx mornyx commented Jul 25, 2023

close: #1139

Signed-off-by: mornyx <mornyx.z@gmail.com>
@netlify
Copy link

netlify bot commented Jul 25, 2023

Deploy Preview for tidb-dashboard-for-dbaas failed.

Name Link
🔨 Latest commit bb8dea0
🔍 Latest deploy log https://app.netlify.com/sites/tidb-dashboard-for-dbaas/deploys/64bf31adf1a2db0007fd823c

@netlify
Copy link

netlify bot commented Jul 25, 2023

Deploy Preview for tidb-dashboard failed.

Name Link
🔨 Latest commit bb8dea0
🔍 Latest deploy log https://app.netlify.com/sites/tidb-dashboard/deploys/64bf31ad5c3ddf000857206a

@ti-chi-bot ti-chi-bot bot requested a review from Renkai July 25, 2023 02:21
@ti-chi-bot ti-chi-bot bot added the size/M label Jul 25, 2023
@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #1562 (bb8dea0) into master (fe7f01f) will increase coverage by 15.69%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1562       +/-   ##
===========================================
+ Coverage    9.03%   24.72%   +15.69%     
===========================================
  Files         122      168       +46     
  Lines       12708    14998     +2290     
===========================================
+ Hits         1148     3709     +2561     
+ Misses      11439    11018      -421     
- Partials      121      271      +150     
Flag Coverage Δ
backend_integration 9.03% <ø> (ø)
backend_ut 27.24% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe7f01f...bb8dea0. Read the comment docs.

@ti-chi-bot ti-chi-bot bot added the lgtm label Jul 26, 2023
@ti-chi-bot
Copy link
Contributor

ti-chi-bot bot commented Jul 26, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baurine

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot
Copy link
Contributor

ti-chi-bot bot commented Jul 26, 2023

[LGTM Timeline notifier]

Timeline:

  • 2023-07-26 03:01:28.620695242 +0000 UTC m=+323431.215226217: ☑️ agreed by baurine.

@ti-chi-bot ti-chi-bot bot added the approved label Jul 26, 2023
@ti-chi-bot ti-chi-bot bot merged commit 22bb50e into pingcap:master Jul 26, 2023
10 of 18 checks passed
@mornyx mornyx deleted the issue1139 branch July 26, 2023 13:32
baurine added a commit that referenced this pull request Aug 16, 2023
* chore: fix prometheus typo (#1558)

* fix(manual-profiling): fix crash caused by undefined filed `raw_data_type` (#1559)

* chore: remove slow query v2 pages (#1530)

* chore: remove slow query v2 pages

* chore: remove @antv/g2plot dep

* tweak: allows custom names to pass tls verification (#1562)

* tweak: allows prom addr with http path (#1563)

* fix: add endpoint verification in debugapi to avoid ssrf (#1564)

* feat: allows ngm timeout to be configured (#1569)

* fix(statements): fix download statements failed issue

* update version to 2023.08.15.1

---------

Co-authored-by: Yexiang Zhang <mornyx.z@gmail.com>
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.

TLS origination to TiDB is not compatible with loopback-bound forwarding proxy
2 participants