Skip to content

解决 Windows 无法启动问题 & 增加企业微信工作台单点登录功能。#16

Open
zlbj1984 wants to merge 18 commits intoTencentCloudADP:mainfrom
zlbj1984:main
Open

解决 Windows 无法启动问题 & 增加企业微信工作台单点登录功能。#16
zlbj1984 wants to merge 18 commits intoTencentCloudADP:mainfrom
zlbj1984:main

Conversation

@zlbj1984
Copy link
Copy Markdown

@zlbj1984 zlbj1984 commented Dec 5, 2025

  1. 解决 Windows 无法启动问题,以便在 Windows 环境中也能进行调试;
  2. 增加企业微信工作台单点登录功能:
    • 在没有 Provider 时,使用用户名密码认证
    • 在只有一个 Provider 时,直接跳转到单点登录URL,无需再点击按钮
    • 在多个 Provider 时,需要点击按钮选择单点登录方式

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 11, 2025

@zlbj1984 is attempting to deploy a commit to the tencentcloudadpdevrel Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Contributor

@liangchen-harold liangchen-harold left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution via PR. I have reviewed it and there are some issues that need to be revised or clarified. We look forward to seeing your reply.

Comment thread server/app_factory.py
if hasattr(time, 'tzset'):
time.tzset()

# Windows compatibility: provide os.getpgid and os.killpg if not available
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is getpgid/killpg really necessary?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no plan to offer the way to login with email/password

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no plan to offer the way to login with email/password

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no plan to offer the way to login with email/password

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no plan to offer the way to login with email/password

Comment thread server/core/oauth.py Outdated
user_detail_api = 'https://qyapi.weixin.qq.com/cgi-bin/user/get'
async with aiohttp.ClientSession() as session:
# 获取`Access Token`
print('获取`Access Token`')
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should use logging.info or logging.debug instead of print

@liangchen-harold
Copy link
Copy Markdown
Contributor

@zlbj1984 We should address only one problem in each PR. Could you please move the new commit 9c318f0 to another branch/PR?

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