doris icon indicating copy to clipboard operation
doris copied to clipboard

[Enhancement](HttpServer) Support https interface

Open yongjinhou opened this issue 2 years ago • 9 comments

Proposed changes

Issue Number: close #15658

Problem summary

  1. Organize http documents
  2. Add http interface authentication for FE
  3. Support https interface for FE
  4. Provide authentication interface
  5. Add http interface authentication for BE
  6. Support https interface for BE

Checklist(Required)

  • [x] Does it affect the original behavior
  • [ ] Has unit tests been added
  • [x] Has document been added or modified
  • [ ] Does it need to update dependencies
  • [x] Is this PR support rollback (If NO, please explain WHY)

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

yongjinhou avatar Feb 16 '23 10:02 yongjinhou

run compile

yongjinhou avatar Feb 22 '23 08:02 yongjinhou

run compile

yongjinhou avatar Feb 22 '23 09:02 yongjinhou

run buildall

yongjinhou avatar Feb 23 '23 02:02 yongjinhou

run p0

yongjinhou avatar Feb 23 '23 06:02 yongjinhou

run p0

yongjinhou avatar Feb 23 '23 08:02 yongjinhou

run compile

yongjinhou avatar Feb 24 '23 15:02 yongjinhou

run buildall

yongjinhou avatar Mar 03 '23 02:03 yongjinhou

LocalTest is being tested.

yongjinhou avatar Mar 03 '23 02:03 yongjinhou

run buildall

yongjinhou avatar Mar 03 '23 04:03 yongjinhou

run buildall

yongjinhou avatar Mar 23 '23 02:03 yongjinhou

run buildall

yongjinhou avatar Mar 23 '23 03:03 yongjinhou

run p0

yongjinhou avatar Mar 23 '23 08:03 yongjinhou

run buildall

yongjinhou avatar Mar 23 '23 10:03 yongjinhou

run p1

yongjinhou avatar Mar 23 '23 12:03 yongjinhou

run p0

yongjinhou avatar Mar 24 '23 01:03 yongjinhou

run p0

yongjinhou avatar Mar 24 '23 04:03 yongjinhou

run p0

yongjinhou avatar Mar 24 '23 06:03 yongjinhou

run p1

yongjinhou avatar Mar 24 '23 09:03 yongjinhou

run buildall

yongjinhou avatar Mar 30 '23 11:03 yongjinhou

run buildall

yongjinhou avatar Mar 30 '23 12:03 yongjinhou

I think it is ok to use 2 ports now. But we should only enable one of them at same time. And user can choose to use which one.

Currently, if enable_https is false, only http is supported. If enable_https is true, both http and https are supported and http is redirected to https. This ensures that the previous link is still available, and that users, promethous, spark/flink-doris-connector, etc., can access using http without changing the protocol used.

yongjinhou avatar Apr 02 '23 02:04 yongjinhou

PR approved by at least one committer and no changes requested.

github-actions[bot] avatar Apr 03 '23 01:04 github-actions[bot]

PR approved by anyone and no changes requested.

github-actions[bot] avatar Apr 03 '23 01:04 github-actions[bot]

run buildall

morningman avatar Apr 03 '23 01:04 morningman

run buildall

yongjinhou avatar Apr 03 '23 02:04 yongjinhou