perf: update release to latest

v4.0.0

.dockerignore

@@ -8,6 +8,3 @@ celerybeat.pid
 .vagrant/
 apps/xpack/.git
 .history/
-.idea
-.venv/
-.env

.gitattributes

@@ -0,0 +1,4 @@
+*.mmdb filter=lfs diff=lfs merge=lfs -text
+*.mo filter=lfs diff=lfs merge=lfs -text
+*.ipdb filter=lfs diff=lfs merge=lfs -text
+

.github/dependabot.yml

@@ -1,10 +0,0 @@
-version: 2
-updates:
-  - package-ecosystem: "uv"
-    directory: "/"
-    schedule:
-      interval: "weekly"
-      day: "monday"
-      time: "09:30"
-      timezone: "Asia/Shanghai"
-    target-branch: dev

.github/workflows/build-base-image.yml

@@ -1,74 +0,0 @@
-name: Build and Push Base Image
-
-on:
-  pull_request:
-    branches:
-      - 'dev'
-      - 'v*'
-    paths:
-      - poetry.lock
-      - pyproject.toml
-      - Dockerfile-base
-      - package.json
-      - go.mod
-      - yarn.lock
-      - pom.xml
-      - install_deps.sh
-      - utils/clean_site_packages.sh
-    types:
-      - opened
-      - synchronize
-      - reopened
-
-jobs:
-  build-and-push:
-    runs-on: ubuntu-22.04
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          ref: ${{ github.event.pull_request.head.ref }}
-
-      - name: Set up QEMU
-        uses: docker/setup-qemu-action@v3
-        with:
-          image: tonistiigi/binfmt:qemu-v7.0.0-28
-
-      - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3
-
-      - name: Login to DockerHub
-        uses: docker/login-action@v2
-        with:
-          username: ${{ secrets.DOCKERHUB_USERNAME }}
-          password: ${{ secrets.DOCKERHUB_TOKEN }}
-
-      - name: Extract date
-        id: vars
-        run: echo "IMAGE_TAG=$(date +'%Y%m%d_%H%M%S')" >> $GITHUB_ENV
-
-      - name: Extract repository name
-        id: repo
-        run: echo "REPO=$(basename ${{ github.repository }})" >> $GITHUB_ENV
-
-      - name: Build and push multi-arch image
-        uses: docker/build-push-action@v6
-        with:
-          platforms: linux/amd64,linux/arm64
-          push: true
-          file: Dockerfile-base
-          tags: jumpserver/core-base:${{ env.IMAGE_TAG }}
-
-      - name: Update Dockerfile
-        run: |
-          sed -i 's|-base:.* AS stage-build|-base:${{ env.IMAGE_TAG }} AS stage-build|' Dockerfile
-
-      - name: Commit changes
-        run: |
-          git config --global user.name 'github-actions[bot]'
-          git config --global user.email 'github-actions[bot]@users.noreply.github.com'
-          git add Dockerfile
-          git commit -m "perf: Update Dockerfile with new base image tag"
-          git push origin ${{ github.event.pull_request.head.ref }}
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/check-compilemessages.yml

@@ -1,31 +0,0 @@
-name: Check I18n files CompileMessages
-
-on:
-  pull_request:
-    branches:
-      - 'dev'
-    paths:
-      - 'apps/i18n/core/**/*.po'
-    types:
-      - opened
-      - synchronize
-      - reopened
-jobs:
-  compile-messages-check:
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-
-      - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3
-
-      - name: Build and check compilemessages
-        uses: docker/build-push-action@v6
-        with:
-          platforms: linux/amd64
-          push: false
-          file: Dockerfile
-          target: stage-build
-          tags: jumpserver/core:stage-build

.github/workflows/discord-release.yml

@@ -1,24 +0,0 @@
-name: Publish Release to Discord
-
-on:
-  release:
-    types: [published]
-
-jobs:
-  send_discord_notification:
-    runs-on: ubuntu-latest
-    if: startsWith(github.event.release.tag_name, 'v4.')
-    steps:
-      - name: Send release notification to Discord
-        env:
-          WEBHOOK_URL: ${{ secrets.DISCORD_CHANGELOG_WEBHOOK }}
-        run: |
-          # 获取标签名称和 release body
-          TAG_NAME="${{ github.event.release.tag_name }}"
-          RELEASE_BODY="${{ github.event.release.body }}"
-
-          # 使用 jq 构建 JSON 数据，以确保安全传递
-          JSON_PAYLOAD=$(jq -n --arg tag "# JumpServer $TAG_NAME Released! 🚀" --arg body "$RELEASE_BODY" '{content: "\($tag)\n\($body)"}')
-
-          # 使用 curl 发送 JSON 数据
-          curl -X POST -H "Content-Type: application/json" -d "$JSON_PAYLOAD" "$WEBHOOK_URL"

.github/workflows/docs-release.yml

@@ -1,24 +0,0 @@
-name: Auto update docs changelog
-
-on:
-  release:
-    types: [published]
-
-jobs:
-  update_docs_changelog:
-    runs-on: ubuntu-latest
-    if: startsWith(github.event.release.tag_name, 'v4.')
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-      - name: Update docs changelog
-        env:
-          TAG_NAME: ${{ github.event.release.tag_name }}
-          DOCS_TOKEN: ${{ secrets.DOCS_TOKEN }}
-        run: |
-          git config --global user.name 'BaiJiangJie'
-          git config --global user.email 'jiangjie.bai@fit2cloud.com'
-
-          git clone https://$DOCS_TOKEN@github.com/jumpservice/documentation.git
-          cd documentation/utils
-          bash update_changelog.sh

.github/workflows/llm-code-review.yml.bak

@@ -1,28 +0,0 @@
-name: LLM Code Review
-
-permissions:
-  contents: read
-  pull-requests: write
-
-on:
-  pull_request:
-    types: [opened, reopened, synchronize]
-
-jobs:
-  llm-code-review:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: fit2cloud/LLM-CodeReview-Action@main
-        env:
-          GITHUB_TOKEN: ${{ secrets.FIT2CLOUDRD_LLM_CODE_REVIEW_TOKEN }}
-          OPENAI_API_KEY: ${{ secrets.ALIYUN_LLM_API_KEY }}
-          LANGUAGE: English
-          OPENAI_API_ENDPOINT: https://dashscope.aliyuncs.com/compatible-mode/v1
-          MODEL: qwen2-1.5b-instruct
-          PROMPT: "Please check the following code differences for any irregularities, potential issues, or optimization suggestions, and provide your answers in English."
-          top_p: 1
-          temperature: 1
-          # max_tokens: 10000
-          MAX_PATCH_LENGTH: 10000 
-          IGNORE_PATTERNS: "/node_modules,*.md,/dist,/.github"
-          FILE_PATTERNS: "*.java,*.go,*.py,*.vue,*.ts,*.js,*.css,*.scss,*.html"

.github/workflows/translate-readme.yml

@@ -1,45 +0,0 @@
-name: Translate README
-on:
-  workflow_dispatch:
-    inputs:
-      source_readme:
-        description: "Source README"
-        required: false
-        default: "./readmes/README.en.md"
-      target_langs:
-        description: "Target Languages"
-        required: false
-        default: "zh-hans,zh-hant,ja,pt-br,es,ru"
-      gen_dir_path:
-        description: "Generate Dir Name"
-        required: false
-        default: "readmes/"
-      push_branch:
-        description: "Push Branch"
-        required: false
-        default: "pr@dev@translate_readme"
-      prompt:
-        description: "AI Translate Prompt"
-        required: false
-        default: ""
-
-      gpt_mode:
-        description: "GPT Mode"
-        required: false
-        default: "gpt-4o-mini"
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Auto Translate
-        uses: jumpserver-dev/action-translate-readme@main
-        env:
-          GITHUB_TOKEN: ${{ secrets.PRIVATE_TOKEN }}
-          OPENAI_API_KEY: ${{ secrets.GPT_API_TOKEN }}
-          GPT_MODE: ${{ github.event.inputs.gpt_mode }}
-          SOURCE_README: ${{ github.event.inputs.source_readme }}
-          TARGET_LANGUAGES: ${{ github.event.inputs.target_langs }}
-          PUSH_BRANCH: ${{ github.event.inputs.push_branch }}
-          GEN_DIR_PATH: ${{ github.event.inputs.gen_dir_path }}
-          PROMPT: ${{ github.event.inputs.prompt }}

.gitignore

@@ -45,7 +45,4 @@ test.py
 .history/
 .test/
 *.mo
-apps.iml
-*.db
-*.mmdb
-*.ipdb
+

.isort.cfg

@@ -1,4 +1,3 @@
 [settings]
 line_length=120
 known_first_party=common,users,assets,perms,authentication,jumpserver,notification,ops,orgs,rbac,settings,terminal,tickets
-

.pylintrc

@@ -1,2 +0,0 @@
-[MESSAGES CONTROL]
-disable=missing-module-docstring,missing-class-docstring,missing-function-docstring,too-many-ancestors

Dockerfile

@@ -1,25 +1,101 @@
-FROM jumpserver/core-base:20250427_062456 AS stage-build
+FROM debian:bullseye-slim as stage-1
+ARG TARGETARCH
+
+ARG DEPENDENCIES="                    \
+        ca-certificates               \
+        wget"
+
+ARG APT_MIRROR=http://mirrors.ustc.edu.cn
+RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,id=core \
+    --mount=type=cache,target=/var/lib/apt,sharing=locked,id=core \
+    set -ex \
+    && rm -f /etc/apt/apt.conf.d/docker-clean \
+    && echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' >/etc/apt/apt.conf.d/keep-cache \
+    && sed -i "s@http://.*.debian.org@${APT_MIRROR}@g" /etc/apt/sources.list \
+    && apt-get update \
+    && apt-get -y install --no-install-recommends ${DEPENDENCIES} \
+    && echo "no" | dpkg-reconfigure dash
+
+WORKDIR /opt
+
+ARG CHECK_VERSION=v1.0.2
+RUN set -ex \
+    && wget https://github.com/jumpserver-dev/healthcheck/releases/download/${CHECK_VERSION}/check-${CHECK_VERSION}-linux-${TARGETARCH}.tar.gz \
+    && tar -xf check-${CHECK_VERSION}-linux-${TARGETARCH}.tar.gz \
+    && mv check /usr/local/bin/ \
+    && chown root:root /usr/local/bin/check \
+    && chmod 755 /usr/local/bin/check \
+    && rm -f check-${CHECK_VERSION}-linux-${TARGETARCH}.tar.gz
+
+ARG RECEPTOR_VERSION=v1.4.5
+RUN set -ex \
+    && wget -O /opt/receptor.tar.gz https://github.com/ansible/receptor/releases/download/${RECEPTOR_VERSION}/receptor_${RECEPTOR_VERSION/v/}_linux_${TARGETARCH}.tar.gz \
+    && tar -xf /opt/receptor.tar.gz -C /usr/local/bin/ \
+    && chown root:root /usr/local/bin/receptor \
+    && chmod 755 /usr/local/bin/receptor \
+    && rm -f /opt/receptor.tar.gz
 
 ARG VERSION
 
 WORKDIR /opt/jumpserver
-
 ADD . .
-
 RUN echo > /opt/jumpserver/config.yml \
     && \
     if [ -n "${VERSION}" ]; then \
         sed -i "s@VERSION = .*@VERSION = '${VERSION}'@g" apps/jumpserver/const.py; \
     fi
 
+FROM python:3.11-slim-bullseye as stage-2
+ARG TARGETARCH
+
+ARG BUILD_DEPENDENCIES="              \
+        g++                           \
+        make                          \
+        pkg-config"
+
+ARG DEPENDENCIES="                    \
+        default-libmysqlclient-dev    \
+        freetds-dev                   \
+        gettext                       \
+        libkrb5-dev                   \
+        libldap2-dev                  \
+        libsasl2-dev"
+
+ARG APT_MIRROR=http://mirrors.ustc.edu.cn
+RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,id=core \
+    --mount=type=cache,target=/var/lib/apt,sharing=locked,id=core \
+    set -ex \
+    && rm -f /etc/apt/apt.conf.d/docker-clean \
+    && echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' >/etc/apt/apt.conf.d/keep-cache \
+    && sed -i "s@http://.*.debian.org@${APT_MIRROR}@g" /etc/apt/sources.list \
+    && apt-get update \
+    && apt-get -y install --no-install-recommends ${BUILD_DEPENDENCIES} \
+    && apt-get -y install --no-install-recommends ${DEPENDENCIES} \
+    && echo "no" | dpkg-reconfigure dash
+
+WORKDIR /opt/jumpserver
+
+ARG PIP_MIRROR=https://pypi.tuna.tsinghua.edu.cn/simple
+RUN --mount=type=cache,target=/root/.cache,sharing=locked,id=core \
+    --mount=type=bind,source=poetry.lock,target=poetry.lock \
+    --mount=type=bind,source=pyproject.toml,target=pyproject.toml \
+    set -ex \
+    && python3 -m venv /opt/py3 \
+    && pip install poetry -i ${PIP_MIRROR} \
+    && poetry config virtualenvs.create false \
+    && . /opt/py3/bin/activate \
+    && poetry install --only main
+
+COPY --from=stage-1 /opt/jumpserver /opt/jumpserver
+
 RUN set -ex \
     && export SECRET_KEY=$(head -c100 < /dev/urandom | base64 | tr -dc A-Za-z0-9 | head -c 48) \
     && . /opt/py3/bin/activate \
     && cd apps \
     && python manage.py compilemessages
 
-
 FROM python:3.11-slim-bullseye
+ARG TARGETARCH
 ENV LANG=en_US.UTF-8 \
     PATH=/opt/py3/bin:$PATH
 
@@ -28,35 +104,38 @@ ARG DEPENDENCIES="                    \
         libx11-dev"
 
 ARG TOOLS="                           \
-        cron                          \
         ca-certificates               \
         default-libmysqlclient-dev    \
         openssh-client                \
         sshpass                       \
         bubblewrap"
 
-ARG APT_MIRROR=http://deb.debian.org
-
-RUN set -ex \
+ARG APT_MIRROR=http://mirrors.ustc.edu.cn
+RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,id=core \
+    --mount=type=cache,target=/var/lib/apt,sharing=locked,id=core \
+    set -ex \
+    && rm -f /etc/apt/apt.conf.d/docker-clean \
+    && echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' >/etc/apt/apt.conf.d/keep-cache \
     && sed -i "s@http://.*.debian.org@${APT_MIRROR}@g" /etc/apt/sources.list \
     && ln -sf /usr/share/zoneinfo/Asia/Shanghai /etc/localtime \
-    && apt-get update > /dev/null \
+    && apt-get update \
     && apt-get -y install --no-install-recommends ${DEPENDENCIES} \
     && apt-get -y install --no-install-recommends ${TOOLS} \
     && mkdir -p /root/.ssh/ \
     && echo "Host *\n\tStrictHostKeyChecking no\n\tUserKnownHostsFile /dev/null\n\tCiphers +aes128-cbc\n\tKexAlgorithms +diffie-hellman-group1-sha1\n\tHostKeyAlgorithms +ssh-rsa" > /root/.ssh/config \
     && echo "no" | dpkg-reconfigure dash \
-    && apt-get clean all \
-    && rm -rf /var/lib/apt/lists/* \
-    && echo "0 3 * * * root find /tmp -type f -mtime +1 -size +1M -exec rm -f {} \; && date > /tmp/clean.log" > /etc/cron.d/cleanup_tmp \
-    && chmod 0644 /etc/cron.d/cleanup_tmp
+    && sed -i "s@# export @export @g" ~/.bashrc \
+    && sed -i "s@# alias @alias @g" ~/.bashrc
 
-COPY --from=stage-build /opt /opt
-COPY --from=stage-build /usr/local/bin /usr/local/bin
-COPY --from=stage-build /opt/jumpserver/apps/libs/ansible/ansible.cfg /etc/ansible/
+COPY --from=stage-2 /opt /opt
+COPY --from=stage-1 /usr/local/bin /usr/local/bin
+COPY --from=stage-1 /opt/jumpserver/apps/libs/ansible/ansible.cfg /etc/ansible/
 
 WORKDIR /opt/jumpserver
 
+ARG VERSION
+ENV VERSION=$VERSION
+
 VOLUME /opt/jumpserver/data
 
 ENTRYPOINT ["./entrypoint.sh"]

Dockerfile-base

@@ -1,61 +0,0 @@
-FROM python:3.11-slim-bullseye
-ARG TARGETARCH
-COPY --from=ghcr.io/astral-sh/uv:0.6.14 /uv /uvx /usr/local/bin/
-# Install APT dependencies
-ARG DEPENDENCIES="                    \
-        ca-certificates               \
-        wget                          \
-        g++                           \
-        make                          \
-        pkg-config                    \
-        default-libmysqlclient-dev    \
-        freetds-dev                   \
-        gettext                       \
-        libkrb5-dev                   \
-        libldap2-dev                  \
-        libsasl2-dev"
-
-ARG APT_MIRROR=http://deb.debian.org
-
-RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,id=core \
-    --mount=type=cache,target=/var/lib/apt,sharing=locked,id=core \
-    set -ex \
-    && rm -f /etc/apt/apt.conf.d/docker-clean \
-    && echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' > /etc/apt/apt.conf.d/keep-cache \
-    && sed -i "s@http://.*.debian.org@${APT_MIRROR}@g" /etc/apt/sources.list \
-    && apt-get update > /dev/null \
-    && apt-get -y install --no-install-recommends ${DEPENDENCIES} \
-    && echo "no" | dpkg-reconfigure dash
-
-# Install bin tools
-ARG CHECK_VERSION=v1.0.4
-RUN set -ex \
-    && wget https://github.com/jumpserver-dev/healthcheck/releases/download/${CHECK_VERSION}/check-${CHECK_VERSION}-linux-${TARGETARCH}.tar.gz \
-    && tar -xf check-${CHECK_VERSION}-linux-${TARGETARCH}.tar.gz \
-    && mv check /usr/local/bin/ \
-    && chown root:root /usr/local/bin/check \
-    && chmod 755 /usr/local/bin/check \
-    && rm -f check-${CHECK_VERSION}-linux-${TARGETARCH}.tar.gz
-
-# Install Python dependencies
-WORKDIR /opt/jumpserver
-
-ARG PIP_MIRROR=https://pypi.org/simple
-ENV POETRY_PYPI_MIRROR_URL=${PIP_MIRROR}
-ENV ANSIBLE_COLLECTIONS_PATHS=/opt/py3/lib/python3.11/site-packages/ansible_collections
-ENV LANG=en_US.UTF-8 \
-    PATH=/opt/py3/bin:$PATH
-
-ENV UV_LINK_MODE=copy
-
-RUN --mount=type=cache,target=/root/.cache \
-    --mount=type=bind,source=pyproject.toml,target=pyproject.toml \
-    --mount=type=bind,source=requirements/clean_site_packages.sh,target=clean_site_packages.sh \
-    --mount=type=bind,source=requirements/collections.yml,target=collections.yml \
-    --mount=type=bind,source=requirements/static_files.sh,target=utils/static_files.sh \
-    set -ex \
-    && uv venv \
-    && uv pip install -i${PIP_MIRROR} -r pyproject.toml \
-    && ln -sf $(pwd)/.venv /opt/py3 \
-    && bash utils/static_files.sh \
-    && bash clean_site_packages.sh

Dockerfile-ee

@@ -1,12 +1,38 @@
-ARG VERSION=dev
+ARG VERSION
 
-FROM registry.fit2cloud.com/jumpserver/xpack:${VERSION} AS build-xpack
-FROM jumpserver/core:${VERSION}-ce
+FROM registry.fit2cloud.com/jumpserver/xpack:${VERSION} as build-xpack
+FROM python:3.11-slim-bullseye as build-core
+ARG BUILD_DEPENDENCIES="              \
+        g++"
 
-COPY --from=build-xpack /opt/xpack /opt/jumpserver/apps/xpack
+ARG APT_MIRROR=http://mirrors.ustc.edu.cn
+RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,id=core \
+    --mount=type=cache,target=/var/lib/apt,sharing=locked,id=core \
+    set -ex \
+    && rm -f /etc/apt/apt.conf.d/docker-clean \
+    && echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' >/etc/apt/apt.conf.d/keep-cache \
+    && sed -i "s@http://.*.debian.org@${APT_MIRROR}@g" /etc/apt/sources.list \
+    && apt-get update \
+    && apt-get -y install --no-install-recommends ${BUILD_DEPENDENCIES} \
+    && echo "no" | dpkg-reconfigure dash
+
+WORKDIR /opt/jumpserver
+
+ARG PIP_MIRROR=https://pypi.tuna.tsinghua.edu.cn/simple
+RUN --mount=type=cache,target=/root/.cache,sharing=locked,id=core \
+    --mount=type=bind,source=poetry.lock,target=/opt/jumpserver/poetry.lock \
+    --mount=type=bind,source=pyproject.toml,target=/opt/jumpserver/pyproject.toml \
+    set -ex \
+    && python3 -m venv /opt/py3 \
+    && pip install poetry -i ${PIP_MIRROR} \
+    && poetry config virtualenvs.create false \
+    && . /opt/py3/bin/activate \
+    && poetry install --only xpack
+
+FROM registry.fit2cloud.com/jumpserver/core:${VERSION}-ce
+ARG TARGETARCH
 
 ARG TOOLS="                           \
-        g++                           \
         curl                          \
         iputils-ping                  \
         netcat-openbsd                \
@@ -15,16 +41,12 @@ ARG TOOLS="                           \
         vim                           \
         wget"
 
-RUN set -ex \
+ARG APT_MIRROR=http://mirrors.ustc.edu.cn
+RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,id=core \
+    --mount=type=cache,target=/var/lib/apt,sharing=locked,id=core \
+    set -ex \
     && apt-get update \
-    && apt-get -y install --no-install-recommends ${TOOLS} \
-    && apt-get clean all \
-    && rm -rf /var/lib/apt/lists/*
-
-WORKDIR /opt/jumpserver
-
-ARG PIP_MIRROR=https://pypi.org/simple
-
-RUN set -ex \
-    && uv pip install -i${PIP_MIRROR} --group xpack
+    && apt-get -y install --no-install-recommends ${TOOLS}
 
+COPY --from=build-core /opt/py3 /opt/py3
+COPY --from=build-xpack /opt/xpack /opt/jumpserver/apps/xpack

GITSHA

@@ -0,0 +1 @@
+02fc045370b39a2d14561ea978d465e162a53119

README.md

@@ -1,62 +1,38 @@
 <div align="center">
   <a name="readme-top"></a>
-  <a href="https://jumpserver.com" target="_blank"><img src="https://download.jumpserver.org/images/jumpserver-logo.svg" alt="JumpServer" width="300" /></a>
+  <a href="https://jumpserver.org/index-en.html"><img src="https://download.jumpserver.org/images/jumpserver-logo.svg" alt="JumpServer" width="300" /></a>
   
 ## An open-source PAM tool (Bastion Host)
 
 [![][license-shield]][license-link]
-[![][docs-shield]][docs-link]
-[![][deepwiki-shield]][deepwiki-link]
 [![][discord-shield]][discord-link]
 [![][docker-shield]][docker-link]
 [![][github-release-shield]][github-release-link]
 [![][github-stars-shield]][github-stars-link]
 
-[English](/README.md) · [中文(简体)](/readmes/README.zh-hans.md) · [中文(繁體)](/readmes/README.zh-hant.md) · [日本語](/readmes/README.ja.md) · [Português (Brasil)](/readmes/README.pt-br.md) · [Español](/readmes/README.es.md) · [Русский](/readmes/README.ru.md)
-
+**English** · [简体中文](./README.zh-CN.md) · [Documents][docs-link] · [Report Bug][github-issues-link] · [Request Feature][github-issues-link]
 </div>
 <br/>
 
 ## What is JumpServer?
 
-JumpServer is an open-source Privileged Access Management (PAM) tool that provides DevOps and IT teams with on-demand and secure access to SSH, RDP, Kubernetes, Database and RemoteApp endpoints through a web browser.
+JumpServer is an open-source Privileged Access Management (PAM) tool that provides DevOps and IT teams with on-demand and secure access to SSH, RDP, Kubernetes, Database and Remote App endpoints through a web browser.
 
-
-<picture>
-  <source media="(prefers-color-scheme: light)" srcset="https://github.com/user-attachments/assets/dd612f3d-c958-4f84-b164-f31b75454d7f">
-  <source media="(prefers-color-scheme: dark)" srcset="https://github.com/user-attachments/assets/28676212-2bc4-4a9f-ae10-3be9320647e3">
-  <img src="https://github.com/user-attachments/assets/dd612f3d-c958-4f84-b164-f31b75454d7f" alt="Theme-based Image">
-</picture>
-
-
-## Quickstart
-
-Prepare a clean Linux Server ( 64 bit, >= 4c8g )
-
-```sh
-curl -sSL https://github.com/jumpserver/jumpserver/releases/latest/download/quick_start.sh | bash
-```
-
-Access JumpServer in your browser at `http://your-jumpserver-ip/`
-- Username: `admin`
-- Password: `ChangeMe`
-
-[![JumpServer Quickstart](https://github.com/user-attachments/assets/0f32f52b-9935-485e-8534-336c63389612)](https://www.youtube.com/watch?v=UlGYRbKrpgY "JumpServer Quickstart")
+![JumpServer Overview](https://github.com/jumpserver/jumpserver/assets/41712985/eb9e6f39-3911-4d4a-bca9-aa50cc3b761d)
 
 ## Screenshots
+
 <table style="border-collapse: collapse; border: 1px solid black;">
   <tr>
     <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/99fabe5b-0475-4a53-9116-4c370a1426c4" alt="JumpServer Console"   /></td>
-    <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/user-attachments/assets/7c1f81af-37e8-4f07-8ac9-182895e1062e" alt="JumpServer PAM"   /></td>    
-  </tr>
-  <tr>
     <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/a424d731-1c70-4108-a7d8-5bbf387dda9a" alt="JumpServer Audits"   /></td>
-    <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/393d2c27-a2d0-4dea-882d-00ed509e00c9" alt="JumpServer Workbench"   /></td>
   </tr>
+
   <tr>
-    <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/user-attachments/assets/eaa41f66-8cc8-4f01-a001-0d258501f1c9" alt="JumpServer RBAC"   /></td>     
+    <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/393d2c27-a2d0-4dea-882d-00ed509e00c9" alt="JumpServer Workbench"   /></td>
     <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/3a2611cd-8902-49b8-b82b-2a6dac851f3e" alt="JumpServer Settings"   /></td>
   </tr>
+
   <tr>
     <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/1e236093-31f7-4563-8eb1-e36d865f1568" alt="JumpServer SSH"   /></td>
     <td style="padding: 5px;background-color:#fff;"><img src= "https://github.com/jumpserver/jumpserver/assets/32935519/69373a82-f7ab-41e8-b763-bbad2ba52167" alt="JumpServer RDP"   /></td>
@@ -67,6 +43,15 @@ Access JumpServer in your browser at `http://your-jumpserver-ip/`
   </tr>
 </table>
 
+## Quick Start
+Prepare a clean Linux Server ( 64bit, >= 4c8g )
+
+```
+curl -sSL https://github.com/jumpserver/jumpserver/releases/latest/download/quick_start.sh | bash
+```
+
+Access JumpServer in your browser at `http://your-ip/`, `admin`/`admin`
+
 ## Components
 
 JumpServer consists of multiple key components, which collectively form the functional framework of JumpServer, providing users with comprehensive capabilities for operations management and security control.
@@ -78,21 +63,26 @@ JumpServer consists of multiple key components, which collectively form the func
 | [KoKo](https://github.com/jumpserver/koko)             | <a href="https://github.com/jumpserver/koko/releases"><img alt="Koko release" src="https://img.shields.io/github/release/jumpserver/koko.svg" /></a>                   | JumpServer Character Protocol Connector                                                                 |
 | [Lion](https://github.com/jumpserver/lion)             | <a href="https://github.com/jumpserver/lion/releases"><img alt="Lion release" src="https://img.shields.io/github/release/jumpserver/lion.svg" /></a>                   | JumpServer Graphical Protocol Connector                                                                 |
 | [Chen](https://github.com/jumpserver/chen)             | <a href="https://github.com/jumpserver/chen/releases"><img alt="Chen release" src="https://img.shields.io/github/release/jumpserver/chen.svg" />                       | JumpServer Web DB                                                                                       |  
-| [Tinker](https://github.com/jumpserver/tinker)         | <img alt="Tinker" src="https://img.shields.io/badge/release-private-red" />                                                                                            | JumpServer Remote Application Connector (Windows)                                                    |
-| [Panda](https://github.com/jumpserver/Panda)           | <img alt="Panda" src="https://img.shields.io/badge/release-private-red" />                                                                                             | JumpServer EE Remote Application Connector (Linux)                                                      |
-| [Razor](https://github.com/jumpserver/razor)           | <img alt="Chen" src="https://img.shields.io/badge/release-private-red" />                                                                                              | JumpServer EE RDP Proxy Connector                                                                       |
-| [Magnus](https://github.com/jumpserver/magnus)         | <img alt="Magnus" src="https://img.shields.io/badge/release-private-red" />                                                                                            | JumpServer EE Database Proxy Connector                                                                  |
-| [Nec](https://github.com/jumpserver/nec)               | <img alt="Nec" src="https://img.shields.io/badge/release-private-red" />                                                                                               | JumpServer EE VNC Proxy Connector                                                                       |
-| [Facelive](https://github.com/jumpserver/facelive)     | <img alt="Facelive" src="https://img.shields.io/badge/release-private-red" />                                                                                          | JumpServer EE Facial Recognition                                                                        |
+| [Razor](https://github.com/jumpserver/razor)           | <img alt="Chen" src="https://img.shields.io/badge/release-private-red" />                                                                                              | JumpServer RDP Proxy Connector                                                                          |
+| [Tinker](https://github.com/jumpserver/tinker)         | <img alt="Tinker" src="https://img.shields.io/badge/release-private-red" />                                                                                            | JumpServer Remote Application Connector (Windows)                                                       |
+| [Panda](https://github.com/jumpserver/Panda)           | <img alt="Panda" src="https://img.shields.io/badge/release-private-red" />                                                                                             | JumpServer Remote Application Connector (Linux)                                                         |
+| [Magnus](https://github.com/jumpserver/magnus)         | <img alt="Magnus" src="https://img.shields.io/badge/release-private-red" />                                                                                            | JumpServer Database Proxy Connector                                                                     |
+
 
 
 ## Contributing
 
 Welcome to submit PR to contribute. Please refer to [CONTRIBUTING.md][contributing-link] for guidelines.
 
+## Security
+
+JumpServer is a mission critical product. Please refer to the Basic Security Recommendations for installation and deployment. If you encounter any security-related issues, please contact us directly:
+
+- Email: support@fit2cloud.com
+
 ## License
 
-Copyright (c) 2014-2025 FIT2CLOUD, All rights reserved.
+Copyright (c) 2014-2024 飞致云 FIT2CLOUD, All rights reserved.
 
 Licensed under The GNU General Public License version 3 (GPLv3) (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at
 
@@ -101,9 +91,8 @@ https://www.gnu.org/licenses/gpl-3.0.html
 Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an " AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
 
 <!-- JumpServer official link -->
-[docs-link]: https://jumpserver.com/docs
-[discord-link]: https://discord.com/invite/W6vYXmAQG2
-[deepwiki-link]: https://deepwiki.com/jumpserver/jumpserver/
+[docs-link]: https://en-docs.jumpserver.org/
+[discord-link]: https://discord.com/invite/jcM5tKWJ
 [contributing-link]: https://github.com/jumpserver/jumpserver/blob/dev/CONTRIBUTING.md
 
 <!-- JumpServer Other link-->
@@ -114,10 +103,10 @@ Unless required by applicable law or agreed to in writing, software distributed
 [github-issues-link]: https://github.com/jumpserver/jumpserver/issues
 
 <!-- Shield link-->
-[docs-shield]: https://img.shields.io/badge/documentation-148F76
 [github-release-shield]: https://img.shields.io/github/v/release/jumpserver/jumpserver
-[github-stars-shield]: https://img.shields.io/github/stars/jumpserver/jumpserver?color=%231890FF&style=flat-square   
+[github-stars-shield]: https://img.shields.io/github/stars/jumpserver/jumpserver?color=%231890FF&style=flat-square
 [docker-shield]: https://img.shields.io/docker/pulls/jumpserver/jms_all.svg
 [license-shield]: https://img.shields.io/github/license/jumpserver/jumpserver
-[deepwiki-shield]: https://img.shields.io/badge/deepwiki-devin?color=blue
 [discord-shield]: https://img.shields.io/discord/1194233267294052363?style=flat&logo=discord&logoColor=%23f5f5f5&labelColor=%235462eb&color=%235462eb
+
+<!-- Image link -->

README.zh-CN.md

@@ -0,0 +1,125 @@
+<p align="center">
+  <a href="https://jumpserver.org"><img src="https://download.jumpserver.org/images/jumpserver-logo.svg" alt="JumpServer" width="300" /></a>
+</p>
+<h3 align="center">广受欢迎的开源堡垒机</h3>
+
+<p align="center">
+  <a href="https://www.gnu.org/licenses/gpl-3.0.html"><img src="https://img.shields.io/github/license/jumpserver/jumpserver" alt="License: GPLv3"></a>
+  <a href="https://hub.docker.com/u/jumpserver"><img src="https://img.shields.io/docker/pulls/jumpserver/jms_all.svg" alt="Docker pulls"></a>
+  <a href="https://github.com/jumpserver/jumpserver/releases/latest"><img src="https://img.shields.io/github/v/release/jumpserver/jumpserver" alt="Latest release"></a>
+  <a href="https://github.com/jumpserver/jumpserver"><img src="https://img.shields.io/github/stars/jumpserver/jumpserver?color=%231890FF&style=flat-square" alt="Stars"></a>
+</p>
+
+
+<p align="center">
+    9 年时间，倾情投入，用心做好一款开源堡垒机。
+</p>
+
+------------------------------
+JumpServer 是广受欢迎的开源堡垒机，是符合 4A 规范的专业运维安全审计系统。
+
+JumpServer 堡垒机帮助企业以更安全的方式管控和登录各种类型的资产，包括：
+
+- **SSH**: Linux / Unix / 网络设备 等；
+- **Windows**: Web 方式连接 / 原生 RDP 连接；
+- **数据库**: MySQL / MariaDB / PostgreSQL / Oracle / SQLServer / ClickHouse 等；
+- **NoSQL**: Redis / MongoDB 等；
+- **GPT**: ChatGPT 等;
+- **云服务**: Kubernetes / VMware vSphere 等;
+- **Web 站点**: 各类系统的 Web 管理后台；
+- **应用**: 通过 Remote App 连接各类应用。
+
+## 产品特色
+
+- **开源**: 零门槛，线上快速获取和安装；
+- **无插件**: 仅需浏览器，极致的 Web Terminal 使用体验；
+- **分布式**: 支持分布式部署和横向扩展，轻松支持大规模并发访问；
+- **多云支持**: 一套系统，同时管理不同云上面的资产；
+- **多租户**: 一套系统，多个子公司或部门同时使用；
+- **云端存储**: 审计录像云端存储，永不丢失；
+
+## UI 展示
+
+![UI展示](https://docs.jumpserver.org/zh/v3/img/dashboard.png)
+
+## 在线体验
+
+- 环境地址：<https://demo.jumpserver.org/>
+
+| :warning: 注意                 |
+|:-----------------------------|
+| 该环境仅作体验目的使用，我们会定时清理、重置数据！    |
+| 请勿修改体验环境用户的密码！               |
+| 请勿在环境中添加业务生产环境地址、用户名密码等敏感信息！ |
+
+## 快速开始
+
+- [快速入门](https://docs.jumpserver.org/zh/v3/quick_start/)
+- [产品文档](https://docs.jumpserver.org)
+- [在线学习](https://edu.fit2cloud.com/page/2635362)
+- [知识库](https://kb.fit2cloud.com/categories/jumpserver)
+
+## 案例研究
+
+- [腾讯音乐娱乐集团：基于JumpServer的安全运维审计解决方案](https://blog.fit2cloud.com/?p=a04cdf0d-6704-4d18-9b40-9180baecd0e2)
+- [腾讯海外游戏：基于JumpServer构建游戏安全运营能力](https://blog.fit2cloud.com/?p=3704)
+- [万华化学：通过JumpServer管理全球化分布式IT资产，并且实现与云管平台的联动](https://blog.fit2cloud.com/?p=3504)
+- [雪花啤酒：JumpServer堡垒机使用体会](https://blog.fit2cloud.com/?p=3412)
+- [顺丰科技：JumpServer 堡垒机护航顺丰科技超大规模资产安全运维](https://blog.fit2cloud.com/?p=1147)
+- [沐瞳游戏：通过JumpServer管控多项目分布式资产](https://blog.fit2cloud.com/?p=3213)
+- [携程：JumpServer 堡垒机部署与运营实战](https://blog.fit2cloud.com/?p=851)
+- [大智慧：JumpServer 堡垒机让“大智慧”的混合 IT 运维更智慧](https://blog.fit2cloud.com/?p=882)
+- [小红书：JumpServer 堡垒机大规模资产跨版本迁移之路](https://blog.fit2cloud.com/?p=516)
+- [中手游：JumpServer堡垒机助力中手游提升多云环境下安全运维能力](https://blog.fit2cloud.com/?p=732)
+- [中通快递：JumpServer主机安全运维实践](https://blog.fit2cloud.com/?p=708)
+- [东方明珠：JumpServer高效管控异构化、分布式云端资产](https://blog.fit2cloud.com/?p=687)
+- [江苏农信：JumpServer堡垒机助力行业云安全运维](https://blog.fit2cloud.com/?p=666)
+
+## 社区交流
+
+如果您在使用过程中有任何疑问或对建议，欢迎提交 [GitHub Issue](https://github.com/jumpserver/jumpserver/issues/new/choose)。
+
+您也可以到我们的 [社区论坛](https://bbs.fit2cloud.com/c/js/5) 当中进行交流沟通。
+
+### 参与贡献
+
+欢迎提交 PR 参与贡献。 参考 [CONTRIBUTING.md](https://github.com/jumpserver/jumpserver/blob/dev/CONTRIBUTING.md)
+
+## 组件项目
+
+| 项目                                                     | 状态                                                                                                                                                                     | 描述                                                                                |
+|--------------------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------|
+| [Lina](https://github.com/jumpserver/lina)             | <a href="https://github.com/jumpserver/lina/releases"><img alt="Lina release" src="https://img.shields.io/github/release/jumpserver/lina.svg" /></a>                   | JumpServer Web UI 项目                                                              |
+| [Luna](https://github.com/jumpserver/luna)             | <a href="https://github.com/jumpserver/luna/releases"><img alt="Luna release" src="https://img.shields.io/github/release/jumpserver/luna.svg" /></a>                   | JumpServer Web Terminal 项目                                                        |
+| [KoKo](https://github.com/jumpserver/koko)             | <a href="https://github.com/jumpserver/koko/releases"><img alt="Koko release" src="https://img.shields.io/github/release/jumpserver/koko.svg" /></a>                   | JumpServer 字符协议 Connector 项目                                                      |
+| [Lion](https://github.com/jumpserver/lion-release)     | <a href="https://github.com/jumpserver/lion-release/releases"><img alt="Lion release" src="https://img.shields.io/github/release/jumpserver/lion-release.svg" /></a>   | JumpServer 图形协议 Connector 项目，依赖 [Apache Guacamole](https://guacamole.apache.org/) |
+| [Razor](https://github.com/jumpserver/razor)           | <img alt="Chen" src="https://img.shields.io/badge/release-私有发布-red" />                                                                                                 | JumpServer RDP 代理 Connector 项目                                                    |
+| [Tinker](https://github.com/jumpserver/tinker)         | <img alt="Tinker" src="https://img.shields.io/badge/release-私有发布-red" />                                                                                               | JumpServer 远程应用 Connector 项目 (Windows)                                                      |
+| [Panda](https://github.com/jumpserver/Panda)         | <img alt="Panda" src="https://img.shields.io/badge/release-私有发布-red" />                                                                                               | JumpServer 远程应用 Connector 项目 (Linux)                                                      |
+| [Magnus](https://github.com/jumpserver/magnus-release) | <a href="https://github.com/jumpserver/magnus-release/releases"><img alt="Magnus release" src="https://img.shields.io/github/release/jumpserver/magnus-release.svg" /> | JumpServer 数据库代理 Connector 项目                                                     |
+| [Chen](https://github.com/jumpserver/chen-release)     | <a href="https://github.com/jumpserver/chen-release/releases"><img alt="Chen release" src="https://img.shields.io/github/release/jumpserver/chen-release.svg" />       | JumpServer Web DB 项目，替代原来的 OmniDB                                                 |
+| [Kael](https://github.com/jumpserver/kael)             | <a href="https://github.com/jumpserver/kael/releases"><img alt="Kael release" src="https://img.shields.io/github/release/jumpserver/kael.svg" />                       | JumpServer 连接 GPT 资产的组件项目                                                         |
+| [Wisp](https://github.com/jumpserver/wisp)             | <a href="https://github.com/jumpserver/wisp/releases"><img alt="Magnus release" src="https://img.shields.io/github/release/jumpserver/wisp.svg" />                     | JumpServer 各系统终端组件和 Core API 通信的组件项目                                              |
+| [Clients](https://github.com/jumpserver/clients)       | <a href="https://github.com/jumpserver/clients/releases"><img alt="Clients release" src="https://img.shields.io/github/release/jumpserver/clients.svg" />              | JumpServer 客户端 项目                                                                 |
+| [Installer](https://github.com/jumpserver/installer)   | <a href="https://github.com/jumpserver/installer/releases"><img alt="Installer release" src="https://img.shields.io/github/release/jumpserver/installer.svg" />        | JumpServer 安装包 项目                                                                 |
+
+## 安全说明
+
+JumpServer是一款安全产品，请参考 [基本安全建议](https://docs.jumpserver.org/zh/master/install/install_security/)
+进行安装部署。如果您发现安全相关问题，请直接联系我们：
+
+- 邮箱：support@fit2cloud.com
+- 电话：400-052-0755
+
+## License & Copyright
+
+Copyright (c) 2014-2024 飞致云 FIT2CLOUD, All rights reserved.
+
+Licensed under The GNU General Public License version 3 (GPLv3)  (the "License"); you may not use this file except in
+compliance with the License. You may obtain a copy of the License at
+
+https://www.gnu.org/licenses/gpl-3.0.html
+
+Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "
+AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific
+language governing permissions and limitations under the License.

SECURITY.md

@@ -5,7 +5,8 @@ JumpServer 是一款正在成长的安全产品， 请参考 [基本安全建议
 如果你发现安全问题，请直接联系我们，我们携手让世界更好：
 
 - ibuler@fit2cloud.com
-- support@lxware.hk
+- support@fit2cloud.com
+- 400-052-0755
 
 
 # Security Policy
@@ -15,5 +16,6 @@ JumpServer is a security product, The installation and development should follow
 All security bugs should be reported to the contact as below:
 
 - ibuler@fit2cloud.com
-- support@lxware.hk
+- support@fit2cloud.com
+- 400-052-0755
 

apps/accounts/api/account/__init__.py

@@ -1,6 +1,4 @@
 from .account import *
-from .application import *
-from .pam_dashboard import *
 from .task import *
 from .template import *
 from .virtual import *

apps/accounts/api/account/account.py

@@ -1,27 +1,20 @@
-from django.db import transaction
 from django.shortcuts import get_object_or_404
-from django.utils.translation import gettext_lazy as _
 from rest_framework.decorators import action
 from rest_framework.generics import ListAPIView, CreateAPIView
 from rest_framework.response import Response
 from rest_framework.status import HTTP_200_OK
 
 from accounts import serializers
-from accounts.const import ChangeSecretRecordStatusChoice
-from accounts.filters import AccountFilterSet, NodeFilterBackend
+from accounts.filters import AccountFilterSet
 from accounts.mixins import AccountRecordViewLogMixin
-from accounts.models import Account, ChangeSecretRecord
+from accounts.models import Account
 from assets.models import Asset, Node
 from authentication.permissions import UserConfirmation, ConfirmType
 from common.api.mixin import ExtraFilterFieldsMixin
-from common.drf.filters import AttrRulesFilterBackend
 from common.permissions import IsValidUser
-from common.utils import lazyproperty, get_logger
 from orgs.mixins.api import OrgBulkModelViewSet
 from rbac.permissions import RBACPermission
 
-logger = get_logger(__file__)
-
 __all__ = [
     'AccountViewSet', 'AccountSecretsViewSet',
     'AccountHistoriesSecretAPI', 'AssetAccountBulkCreateApi',
@@ -31,7 +24,6 @@ __all__ = [
 class AccountViewSet(OrgBulkModelViewSet):
     model = Account
     search_fields = ('username', 'name', 'asset__name', 'asset__address', 'comment')
-    extra_filter_backends = [AttrRulesFilterBackend, NodeFilterBackend]
     filterset_class = AccountFilterSet
     serializer_classes = {
         'default': serializers.AccountSerializer,
@@ -41,21 +33,9 @@ class AccountViewSet(OrgBulkModelViewSet):
         'partial_update': ['accounts.change_account'],
         'su_from_accounts': 'accounts.view_account',
         'clear_secret': 'accounts.change_account',
-        'move_to_assets': 'accounts.create_account',
-        'copy_to_assets': 'accounts.create_account',
     }
     export_as_zip = True
 
-    def get_queryset(self):
-        queryset = super().get_queryset()
-        asset_id = self.request.query_params.get('asset') or self.request.query_params.get('asset_id')
-        if not asset_id:
-            return queryset
-
-        asset = get_object_or_404(Asset, pk=asset_id)
-        queryset = asset.all_accounts.all()
-        return queryset
-
     @action(methods=['get'], detail=False, url_path='su-from-accounts')
     def su_from_accounts(self, request, *args, **kwargs):
         account_id = request.query_params.get('account')
@@ -106,45 +86,6 @@ class AccountViewSet(OrgBulkModelViewSet):
         self.model.objects.filter(id__in=account_ids).update(secret=None)
         return Response(status=HTTP_200_OK)
 
-    def _copy_or_move_to_assets(self, request, move=False):
-        account = self.get_object()
-        asset_ids = request.data.get('assets', [])
-        assets = Asset.objects.filter(id__in=asset_ids)
-        field_names = [
-            'name', 'username', 'secret_type', 'secret',
-            'privileged', 'is_active', 'source', 'source_id', 'comment'
-        ]
-        account_data = {field: getattr(account, field) for field in field_names}
-
-        creation_results = {}
-        success_count = 0
-
-        for asset in assets:
-            account_data['asset'] = asset
-            creation_results[asset] = {'state': 'created'}
-            try:
-                with transaction.atomic():
-                    self.model.objects.create(**account_data)
-                    success_count += 1
-            except Exception as e:
-                logger.debug(f'{"Move" if move else "Copy"} to assets error: {e}')
-                creation_results[asset] = {'error': _('Account already exists'), 'state': 'error'}
-
-        results = [{'asset': str(asset), **res} for asset, res in creation_results.items()]
-
-        if move and success_count > 0:
-            account.delete()
-
-        return Response(results, status=HTTP_200_OK)
-
-    @action(methods=['post'], detail=True, url_path='move-to-assets')
-    def move_to_assets(self, request, *args, **kwargs):
-        return self._copy_or_move_to_assets(request, move=True)
-
-    @action(methods=['post'], detail=True, url_path='copy-to-assets')
-    def copy_to_assets(self, request, *args, **kwargs):
-        return self._copy_or_move_to_assets(request, move=False)
-
 
 class AccountSecretsViewSet(AccountRecordViewLogMixin, AccountViewSet):
     """
@@ -184,31 +125,17 @@ class AccountHistoriesSecretAPI(ExtraFilterFieldsMixin, AccountRecordViewLogMixi
         'GET': 'accounts.view_accountsecret',
     }
 
-    @lazyproperty
-    def account(self) -> Account:
-        return get_object_or_404(Account, pk=self.kwargs.get('pk'))
-
     def get_object(self):
-        return self.account
-
-    @lazyproperty
-    def latest_history(self):
-        return self.account.history.first()
-
-    @property
-    def latest_change_secret_record(self) -> ChangeSecretRecord:
-        return self.account.changesecretrecords.filter(
-            status=ChangeSecretRecordStatusChoice.pending
-        ).order_by('-date_created').first()
+        return get_object_or_404(Account, pk=self.kwargs.get('pk'))
 
     @staticmethod
     def filter_spm_queryset(resource_ids, queryset):
         return queryset.filter(history_id__in=resource_ids)
 
     def get_queryset(self):
-        account = self.account
+        account = self.get_object()
         histories = account.history.all()
-        latest_history = self.latest_history
+        latest_history = account.history.first()
         if not latest_history:
             return histories
         if account.secret != latest_history.secret:
@@ -217,25 +144,3 @@ class AccountHistoriesSecretAPI(ExtraFilterFieldsMixin, AccountRecordViewLogMixi
             return histories
         histories = histories.exclude(history_id=latest_history.history_id)
         return histories
-
-    def filter_queryset(self, queryset):
-        queryset = super().filter_queryset(queryset)
-        queryset = list(queryset)
-        latest_history = self.latest_history
-        if not latest_history:
-            return queryset
-
-        latest_change_secret_record = self.latest_change_secret_record
-        if not latest_change_secret_record:
-            return queryset
-
-        if latest_change_secret_record.date_created > latest_history.history_date:
-            temp_history = self.model(
-                secret=latest_change_secret_record.new_secret,
-                secret_type=self.account.secret_type,
-                version=latest_history.version,
-                history_date=latest_change_secret_record.date_created,
-            )
-            queryset = [temp_history] + queryset
-
-        return queryset

apps/accounts/api/account/application.py

@@ -1,84 +0,0 @@
-import os
-
-from django.conf import settings
-from django.utils.translation import gettext_lazy as _, get_language
-from rest_framework.decorators import action
-from rest_framework.response import Response
-
-from accounts import serializers
-from accounts.models import IntegrationApplication
-from audits.models import IntegrationApplicationLog
-from authentication.permissions import UserConfirmation, ConfirmType
-from common.exceptions import JMSException
-from common.permissions import IsValidUser
-from common.utils import get_request_ip
-from orgs.mixins.api import OrgBulkModelViewSet
-from rbac.permissions import RBACPermission
-
-
-class IntegrationApplicationViewSet(OrgBulkModelViewSet):
-    model = IntegrationApplication
-    search_fields = ('name', 'comment')
-    serializer_classes = {
-        'default': serializers.IntegrationApplicationSerializer,
-        'get_account_secret': serializers.IntegrationAccountSecretSerializer
-    }
-    rbac_perms = {
-        'get_once_secret': 'accounts.change_integrationapplication',
-        'get_account_secret': 'accounts.view_integrationapplication'
-    }
-
-    def read_file(self, path):
-        if os.path.exists(path):
-            with open(path, 'r', encoding='utf-8') as file:
-                return file.read()
-        return ''
-
-    @action(
-        ['GET'], detail=False, url_path='sdks',
-        permission_classes=[IsValidUser]
-    )
-    def get_sdks_info(self, request, *args, **kwargs):
-        code_suffix_mapper = {
-            'python': 'py',
-            'java': 'java',
-            'go': 'go',
-            'node': 'js',
-            'curl': 'sh',
-        }
-        sdk_language = request.query_params.get('language', 'python')
-        sdk_path = os.path.join(settings.APPS_DIR, 'accounts', 'demos', sdk_language)
-        readme_path = os.path.join(sdk_path, f'README.{get_language()}.md')
-        demo_path = os.path.join(sdk_path, f'demo.{code_suffix_mapper[sdk_language]}')
-
-        readme_content = self.read_file(readme_path)
-        demo_content = self.read_file(demo_path)
-
-        return Response(data={'readme': readme_content, 'code': demo_content})
-
-    @action(
-        ['GET'], detail=True, url_path='secret',
-        permission_classes=[RBACPermission, UserConfirmation.require(ConfirmType.MFA)]
-    )
-    def get_once_secret(self, request, *args, **kwargs):
-        instance = self.get_object()
-        return Response(data={'id': instance.id, 'secret': instance.secret})
-
-    @action(['GET'], detail=False, url_path='account-secret',
-            permission_classes=[RBACPermission])
-    def get_account_secret(self, request, *args, **kwargs):
-        serializer = self.get_serializer(data=request.query_params)
-        if not serializer.is_valid():
-            return Response({'error': serializer.errors}, status=400)
-
-        service = request.user
-        account = service.get_account(**serializer.data)
-        if not account:
-            msg = _('Account not found')
-            raise JMSException(code='Not found', detail='%s' % msg)
-        asset = account.asset
-        IntegrationApplicationLog.objects.create(
-            remote_addr=get_request_ip(request), service=service.name, service_id=service.id,
-            account=f'{account.name}({account.username})', asset=f'{asset.name}({asset.address})',
-        )
-        return Response(data={'id': request.user.id, 'secret': account.secret})

apps/accounts/api/account/pam_dashboard.py

@@ -1,130 +0,0 @@
-# -*- coding: utf-8 -*-
-#
-from collections import defaultdict
-
-from django.db.models import Count, F, Q
-from django.http.response import JsonResponse
-from rest_framework.views import APIView
-
-from accounts.models import (
-    Account, GatherAccountsAutomation,
-    PushAccountAutomation, BackupAccountAutomation,
-    AccountRisk, IntegrationApplication, ChangeSecretAutomation
-)
-from assets.const import AllTypes
-from common.utils.timezone import local_monday
-
-__all__ = ['PamDashboardApi']
-
-
-class PamDashboardApi(APIView):
-    http_method_names = ['get']
-    rbac_perms = {
-        'GET': 'accounts.view_account',
-    }
-
-    @staticmethod
-    def get_type_to_accounts():
-        result = Account.objects.annotate(type=F('asset__platform__type')) \
-            .values('type').order_by('type').annotate(total=Count(1))
-        all_types_dict = dict(AllTypes.choices())
-
-        return [
-            {**i, 'label': all_types_dict.get(i['type'], i['type'])}
-            for i in result
-        ]
-
-    @staticmethod
-    def get_account_risk_data(_all, query_params):
-        agg_map = {
-            'total_long_time_no_login_accounts': ('long_time_no_login_count', Q(risk='long_time_no_login')),
-            'total_new_found_accounts': ('new_found_count', Q(risk='new_found')),
-            'total_groups_changed_accounts': ('groups_changed_count', Q(risk='groups_changed')),
-            'total_sudoers_changed_accounts': ('sudoers_changed_count', Q(risk='sudoers_changed')),
-            'total_authorized_keys_changed_accounts': (
-                'authorized_keys_changed_count', Q(risk='authorized_keys_changed')),
-            'total_account_deleted_accounts': ('account_deleted_count', Q(risk='account_deleted')),
-            'total_password_expired_accounts': ('password_expired_count', Q(risk='password_expired')),
-            'total_long_time_password_accounts': ('long_time_password_count', Q(risk='long_time_password')),
-            'total_weak_password_accounts': ('weak_password_count', Q(risk='weak_password')),
-            'total_leaked_password_accounts': ('leaked_password_count', Q(risk='leaked_password')),
-            'total_repeated_password_accounts': ('repeated_password_count', Q(risk='repeated_password')),
-        }
-
-        aggregations = {
-            agg_key: Count('id', distinct=True, filter=agg_filter)
-            for param_key, (agg_key, agg_filter) in agg_map.items()
-            if _all or query_params.get(param_key)
-        }
-
-        data = {}
-        if aggregations:
-            account_stats = AccountRisk.objects.aggregate(**aggregations)
-            data = {param_key: account_stats.get(agg_key) for param_key, (agg_key, _) in agg_map.items() if
-                    agg_key in account_stats}
-
-        return data
-
-    @staticmethod
-    def get_account_data(_all, query_params):
-        agg_map = {
-            'total_accounts': ('total_count', Count('id')),
-            'total_privileged_accounts': ('privileged_count', Count('id', filter=Q(privileged=True))),
-            'total_connectivity_ok_accounts': ('connectivity_ok_count', Count('id', filter=Q(connectivity='ok'))),
-            'total_secret_reset_accounts': ('secret_reset_count', Count('id', filter=Q(secret_reset=True))),
-            'total_valid_accounts': ('valid_count', Count('id', filter=Q(is_active=True))),
-            'total_week_add_accounts': ('week_add_count', Count('id', filter=Q(date_created__gte=local_monday()))),
-        }
-
-        aggregations = {
-            agg_key: agg_expr
-            for param_key, (agg_key, agg_expr) in agg_map.items()
-            if _all or query_params.get(param_key)
-        }
-
-        data = {}
-        account_stats = Account.objects.aggregate(**aggregations)
-        for param_key, (agg_key, __) in agg_map.items():
-            if agg_key in account_stats:
-                data[param_key] = account_stats[agg_key]
-
-        if _all or query_params.get('total_ordinary_accounts'):
-            if 'total_count' in account_stats and 'privileged_count' in account_stats:
-                data['total_ordinary_accounts'] = \
-                    account_stats['total_count'] - account_stats['privileged_count']
-
-        return data
-
-    @staticmethod
-    def get_automation_counts(_all, query_params):
-        automation_counts = defaultdict(int)
-        automation_models = {
-            'total_count_change_secret_automation': ChangeSecretAutomation,
-            'total_count_gathered_account_automation': GatherAccountsAutomation,
-            'total_count_push_account_automation': PushAccountAutomation,
-            'total_count_backup_account_automation': BackupAccountAutomation,
-            'total_count_integration_application': IntegrationApplication,
-        }
-
-        for param_key, model in automation_models.items():
-            if _all or query_params.get(param_key):
-                automation_counts[param_key] = model.objects.count()
-
-        return automation_counts
-
-    def get(self, request, *args, **kwargs):
-        query_params = self.request.query_params
-
-        _all = query_params.get('all')
-
-        data = {}
-        data.update(self.get_account_data(_all, query_params))
-        data.update(self.get_account_risk_data(_all, query_params))
-        data.update(self.get_automation_counts(_all, query_params))
-
-        if _all or query_params.get('total_count_type_to_accounts'):
-            data.update({
-                'total_count_type_to_accounts': self.get_type_to_accounts(),
-            })
-
-        return JsonResponse(data, status=200)

apps/accounts/api/automations/__init__.py

@@ -1,7 +1,5 @@
 from .backup import *
 from .base import *
 from .change_secret import *
-from .change_secret_dashboard import *
-from .check_account import *
-from .gather_account import *
+from .gather_accounts import *
 from .push_account import *

apps/accounts/api/automations/backup.py

@@ -1,36 +1,41 @@
 # -*- coding: utf-8 -*-
 #
+from rest_framework import status, viewsets
+from rest_framework.response import Response
+
 from accounts import serializers
-from accounts.const import AutomationTypes
 from accounts.models import (
-    BackupAccountAutomation
+    AccountBackupAutomation, AccountBackupExecution
 )
+from accounts.tasks import execute_account_backup_task
+from common.const.choices import Trigger
 from orgs.mixins.api import OrgBulkModelViewSet
-from .base import AutomationExecutionViewSet
 
 __all__ = [
-    'BackupAccountViewSet', 'BackupAccountExecutionViewSet'
+    'AccountBackupPlanViewSet', 'AccountBackupPlanExecutionViewSet'
 ]
 
 
-class BackupAccountViewSet(OrgBulkModelViewSet):
-    model = BackupAccountAutomation
+class AccountBackupPlanViewSet(OrgBulkModelViewSet):
+    model = AccountBackupAutomation
     filterset_fields = ('name',)
     search_fields = filterset_fields
-    serializer_class = serializers.BackupAccountSerializer
+    serializer_class = serializers.AccountBackupSerializer
 
 
-class BackupAccountExecutionViewSet(AutomationExecutionViewSet):
-    rbac_perms = (
-        ("list", "accounts.view_backupaccountexecution"),
-        ("retrieve", "accounts.view_backupaccountexecution"),
-        ("create", "accounts.add_backupaccountexecution"),
-        ("report", "accounts.view_backupaccountexecution"),
-    )
-
-    tp = AutomationTypes.backup_account
+class AccountBackupPlanExecutionViewSet(viewsets.ModelViewSet):
+    serializer_class = serializers.AccountBackupPlanExecutionSerializer
+    search_fields = ('trigger', 'plan__name')
+    filterset_fields = ('trigger', 'plan_id', 'plan__name')
+    http_method_names = ['get', 'post', 'options']
 
     def get_queryset(self):
-        queryset = super().get_queryset()
-        queryset = queryset.filter(type=self.tp)
+        queryset = AccountBackupExecution.objects.all()
         return queryset
+
+    def create(self, request, *args, **kwargs):
+        serializer = self.get_serializer(data=request.data)
+        serializer.is_valid(raise_exception=True)
+        pid = serializer.data.get('plan')
+        task = execute_account_backup_task.delay(pid=str(pid), trigger=Trigger.manual)
+        return Response({'task': task.id}, status=status.HTTP_201_CREATED)

apps/accounts/api/automations/base.py

@@ -1,12 +1,8 @@
-from django.http import HttpResponse
 from django.shortcuts import get_object_or_404
 from django.utils.translation import gettext_lazy as _
-from django.views.decorators.clickjacking import xframe_options_sameorigin
 from rest_framework import status, mixins, viewsets
-from rest_framework.decorators import action
 from rest_framework.response import Response
 
-from accounts.filters import AutomationExecutionFilterSet
 from accounts.models import AutomationExecution
 from accounts.tasks import execute_account_automation_task
 from assets import serializers
@@ -17,7 +13,7 @@ from orgs.mixins import generics
 __all__ = [
     'AutomationAssetsListApi', 'AutomationRemoveAssetApi',
     'AutomationAddAssetApi', 'AutomationNodeAddRemoveApi',
-    'AutomationExecutionViewSet'
+    'AutomationExecutionViewSet',
 ]
 
 
@@ -39,10 +35,9 @@ class AutomationAssetsListApi(generics.ListAPIView):
         return assets
 
 
-class AutomationRemoveAssetApi(generics.UpdateAPIView):
+class AutomationRemoveAssetApi(generics.RetrieveUpdateAPIView):
     model = BaseAutomation
     serializer_class = serializers.UpdateAssetSerializer
-    http_method_names = ['patch']
 
     def update(self, request, *args, **kwargs):
         instance = self.get_object()
@@ -57,10 +52,9 @@ class AutomationRemoveAssetApi(generics.UpdateAPIView):
         return Response({'msg': 'ok'})
 
 
-class AutomationAddAssetApi(generics.UpdateAPIView):
+class AutomationAddAssetApi(generics.RetrieveUpdateAPIView):
     model = BaseAutomation
     serializer_class = serializers.UpdateAssetSerializer
-    http_method_names = ['patch']
 
     def update(self, request, *args, **kwargs):
         instance = self.get_object()
@@ -74,10 +68,9 @@ class AutomationAddAssetApi(generics.UpdateAPIView):
             return Response({"error": serializer.errors})
 
 
-class AutomationNodeAddRemoveApi(generics.UpdateAPIView):
+class AutomationNodeAddRemoveApi(generics.RetrieveUpdateAPIView):
     model = BaseAutomation
     serializer_class = serializers.UpdateNodeSerializer
-    http_method_names = ['patch']
 
     def update(self, request, *args, **kwargs):
         action_params = ['add', 'remove']
@@ -104,8 +97,8 @@ class AutomationExecutionViewSet(
 ):
     search_fields = ('trigger', 'automation__name')
     filterset_fields = ('trigger', 'automation_id', 'automation__name')
-    filterset_class = AutomationExecutionFilterSet
     serializer_class = serializers.AutomationExecutionSerializer
+
     tp: str
 
     def get_queryset(self):
@@ -120,10 +113,3 @@ class AutomationExecutionViewSet(
             pid=str(automation.pk), trigger=Trigger.manual, tp=self.tp
         )
         return Response({'task': task.id}, status=status.HTTP_201_CREATED)
-
-    @xframe_options_sameorigin
-    @action(methods=['get'], detail=True, url_path='report')
-    def report(self, request, *args, **kwargs):
-        execution = self.get_object()
-        report = execution.manager.gen_report()
-        return HttpResponse(report)

apps/accounts/api/automations/change_secret.py

@@ -1,17 +1,15 @@
 # -*- coding: utf-8 -*-
 #
-from django.db.models import Max, Q, Subquery, OuterRef
 from rest_framework import status, mixins
 from rest_framework.decorators import action
 from rest_framework.response import Response
 
 from accounts import serializers
-from accounts.const import AutomationTypes, ChangeSecretRecordStatusChoice
+from accounts.const import AutomationTypes
 from accounts.filters import ChangeSecretRecordFilterSet
 from accounts.models import ChangeSecretAutomation, ChangeSecretRecord
 from accounts.tasks import execute_automation_record_task
 from authentication.permissions import UserConfirmation, ConfirmType
-from common.permissions import IsValidLicense
 from orgs.mixins.api import OrgBulkModelViewSet, OrgGenericViewSet
 from rbac.permissions import RBACPermission
 from .base import (
@@ -29,7 +27,6 @@ __all__ = [
 
 class ChangeSecretAutomationViewSet(OrgBulkModelViewSet):
     model = ChangeSecretAutomation
-    permission_classes = [RBACPermission, IsValidLicense]
     filterset_fields = ('name', 'secret_type', 'secret_strategy')
     search_fields = filterset_fields
     serializer_class = serializers.ChangeSecretAutomationSerializer
@@ -37,9 +34,7 @@ class ChangeSecretAutomationViewSet(OrgBulkModelViewSet):
 
 class ChangeSecretRecordViewSet(mixins.ListModelMixin, OrgGenericViewSet):
     filterset_class = ChangeSecretRecordFilterSet
-    permission_classes = [RBACPermission, IsValidLicense]
-    search_fields = ('asset__address', 'account__username')
-    ordering_fields = ('date_finished',)
+    search_fields = ('asset__address',)
     tp = AutomationTypes.change_secret
     serializer_classes = {
         'default': serializers.ChangeSecretRecordSerializer,
@@ -48,8 +43,6 @@ class ChangeSecretRecordViewSet(mixins.ListModelMixin, OrgGenericViewSet):
     rbac_perms = {
         'execute': 'accounts.add_changesecretexecution',
         'secret': 'accounts.view_changesecretrecord',
-        'dashboard': 'accounts.view_changesecretrecord',
-        'ignore_fail': 'accounts.view_changesecretrecord',
     }
 
     def get_permissions(self):
@@ -60,35 +53,8 @@ class ChangeSecretRecordViewSet(mixins.ListModelMixin, OrgGenericViewSet):
             ]
         return super().get_permissions()
 
-    def filter_queryset(self, queryset):
-        queryset = super().filter_queryset(queryset)
-
-        if self.action == 'dashboard':
-            return self.get_dashboard_queryset(queryset)
-        return queryset
-
-    @staticmethod
-    def get_dashboard_queryset(queryset):
-        recent_dates = queryset.values('account').annotate(
-            max_date_finished=Max('date_finished')
-        )
-
-        recent_success_accounts = queryset.filter(
-            account=OuterRef('account'),
-            date_finished=Subquery(
-                recent_dates.filter(account=OuterRef('account')).values('max_date_finished')[:1]
-            )
-        ).filter(Q(status=ChangeSecretRecordStatusChoice.success))
-
-        failed_records = queryset.filter(
-            ~Q(account__in=Subquery(recent_success_accounts.values('account'))),
-            status=ChangeSecretRecordStatusChoice.failed,
-            ignore_fail=False
-        )
-        return failed_records
-
     def get_queryset(self):
-        return ChangeSecretRecord.get_valid_records()
+        return ChangeSecretRecord.objects.all()
 
     @action(methods=['post'], detail=False, url_path='execute')
     def execute(self, request, *args, **kwargs):
@@ -109,31 +75,19 @@ class ChangeSecretRecordViewSet(mixins.ListModelMixin, OrgGenericViewSet):
         serializer = self.get_serializer(instance)
         return Response(serializer.data)
 
-    @action(methods=['get'], detail=False, url_path='dashboard')
-    def dashboard(self, request, *args, **kwargs):
-        return super().list(request, *args, **kwargs)
-
-    @action(methods=['patch'], detail=True, url_path='ignore-fail')
-    def ignore_fail(self, request, *args, **kwargs):
-        instance = self.get_object()
-        instance.ignore_fail = True
-        instance.save(update_fields=['ignore_fail'])
-        return Response(status=status.HTTP_200_OK)
-
 
 class ChangSecretExecutionViewSet(AutomationExecutionViewSet):
     rbac_perms = (
         ("list", "accounts.view_changesecretexecution"),
         ("retrieve", "accounts.view_changesecretexecution"),
         ("create", "accounts.add_changesecretexecution"),
-        ("report", "accounts.view_changesecretexecution"),
     )
-    permission_classes = [RBACPermission, IsValidLicense]
+
     tp = AutomationTypes.change_secret
 
     def get_queryset(self):
         queryset = super().get_queryset()
-        queryset = queryset.filter(type=self.tp)
+        queryset = queryset.filter(automation__type=self.tp)
         return queryset
 
 

apps/accounts/api/automations/change_secret_dashboard.py

@@ -1,185 +0,0 @@
-# -*- coding: utf-8 -*-
-#
-from collections import defaultdict
-
-from django.core.cache import cache
-from django.http.response import JsonResponse
-from django.utils import timezone
-from rest_framework.views import APIView
-
-from accounts.const import AutomationTypes, ChangeSecretRecordStatusChoice
-from accounts.models import ChangeSecretAutomation, AutomationExecution, ChangeSecretRecord
-from assets.models import Node, Asset
-from common.const import Status
-from common.permissions import IsValidLicense
-from common.utils import lazyproperty
-from common.utils.timezone import local_zero_hour, local_now
-from ops.celery import app
-from rbac.permissions import RBACPermission
-
-__all__ = ['ChangeSecretDashboardApi']
-
-
-class ChangeSecretDashboardApi(APIView):
-    http_method_names = ['get']
-    rbac_perms = {
-        'GET': 'accounts.view_changesecretautomation',
-    }
-    permission_classes = [RBACPermission, IsValidLicense]
-    tp = AutomationTypes.change_secret
-    task_name = 'accounts.tasks.automation.execute_account_automation_task'
-    ongoing_change_secret_cache_key = "ongoing_change_secret_cache_key"
-
-    @lazyproperty
-    def days(self):
-        count = self.request.query_params.get('days', 1)
-        return int(count)
-
-    @property
-    def days_to_datetime(self):
-        if self.days == 1:
-            return local_zero_hour()
-        return local_now() - timezone.timedelta(days=self.days)
-
-    def get_queryset_date_filter(self, qs, query_field='date_updated'):
-        return qs.filter(**{f'{query_field}__gte': self.days_to_datetime})
-
-    @lazyproperty
-    def date_range_list(self):
-        return [
-            (local_now() - timezone.timedelta(days=i)).date()
-            for i in range(self.days - 1, -1, -1)
-        ]
-
-    def filter_by_date_range(self, queryset, field_name):
-        date_range_bounds = self.days_to_datetime.date(), (local_now() + timezone.timedelta(days=1)).date()
-        return queryset.filter(**{f'{field_name}__range': date_range_bounds})
-
-    def calculate_daily_metrics(self, queryset, date_field):
-        filtered_queryset = self.filter_by_date_range(queryset, date_field)
-        results = filtered_queryset.values_list(date_field, 'status')
-
-        status_counts = defaultdict(lambda: defaultdict(int))
-
-        for date_finished, status in results:
-            date_str = str(date_finished.date())
-            if status == ChangeSecretRecordStatusChoice.failed:
-                status_counts[date_str]['failed'] += 1
-            elif status == ChangeSecretRecordStatusChoice.success:
-                status_counts[date_str]['success'] += 1
-
-        metrics = defaultdict(list)
-        for date in self.date_range_list:
-            date_str = str(date)
-            for status in ['success', 'failed']:
-                metrics[status].append(status_counts[date_str].get(status, 0))
-
-        return metrics
-
-    def get_daily_success_and_failure_metrics(self):
-        metrics = self.calculate_daily_metrics(self.change_secret_records_queryset, 'date_finished')
-        return metrics.get('success', []), metrics.get('failed', [])
-
-    @lazyproperty
-    def change_secrets_queryset(self):
-        return ChangeSecretAutomation.objects.all()
-
-    @lazyproperty
-    def change_secret_records_queryset(self):
-        return ChangeSecretRecord.get_valid_records()
-
-    def get_change_secret_asset_queryset(self):
-        qs = self.change_secrets_queryset
-        node_ids = qs.filter(nodes__isnull=False).values_list('nodes', flat=True).distinct()
-        nodes = Node.objects.filter(id__in=node_ids)
-        node_asset_ids = Node.get_nodes_all_assets(*nodes).values_list('id', flat=True)
-        direct_asset_ids = qs.filter(assets__isnull=False).values_list('assets', flat=True).distinct()
-        asset_ids = set(list(direct_asset_ids) + list(node_asset_ids))
-        return Asset.objects.filter(id__in=asset_ids)
-
-    def get_filtered_counts(self, qs, field=None):
-        if field is None:
-            return qs.count()
-        return self.get_queryset_date_filter(qs, field).count()
-
-    def get_status_counts(self, executions):
-        executions = executions.filter(type=self.tp)
-        total, failed, success = 0, 0, 0
-        for status in executions.values_list('status', flat=True):
-            total += 1
-            if status in [Status.failed, Status.error]:
-                failed += 1
-            elif status == Status.success:
-                success += 1
-
-        return {
-            'total_count_change_secret_executions': total,
-            'total_count_success_change_secret_executions': success,
-            'total_count_failed_change_secret_executions': failed,
-        }
-
-    def get(self, request, *args, **kwargs):
-        query_params = self.request.query_params
-        data = {}
-
-        _all = query_params.get('all')
-
-        if _all or query_params.get('total_count_change_secrets'):
-            data['total_count_change_secrets'] = self.get_filtered_counts(
-                self.change_secrets_queryset
-            )
-
-        if _all or query_params.get('total_count_periodic_change_secrets'):
-            data['total_count_periodic_change_secrets'] = self.get_filtered_counts(
-                self.change_secrets_queryset.filter(is_periodic=True)
-            )
-
-        if _all or query_params.get('total_count_change_secret_assets'):
-            data['total_count_change_secret_assets'] = self.get_change_secret_asset_queryset().count()
-
-        if _all or query_params.get('total_count_change_secret_status'):
-            executions = self.get_queryset_date_filter(AutomationExecution.objects.all(), 'date_start')
-            data.update(self.get_status_counts(executions))
-
-        if _all or query_params.get('daily_success_and_failure_metrics'):
-            success, failed = self.get_daily_success_and_failure_metrics()
-            data.update({
-                'dates_metrics_date': [date.strftime('%m-%d') for date in self.date_range_list] or ['0'],
-                'dates_metrics_total_count_success': success,
-                'dates_metrics_total_count_failed': failed,
-            })
-
-        if _all or query_params.get('total_count_ongoing_change_secret'):
-            ongoing_counts = cache.get(self.ongoing_change_secret_cache_key)
-            if ongoing_counts is None:
-                execution_ids = []
-                inspect = app.control.inspect()
-                try:
-                    active_tasks = inspect.active()
-                except Exception:
-                    active_tasks = None
-                if active_tasks:
-                    for tasks in active_tasks.values():
-                        for task in tasks:
-                            _id = task.get('id')
-                            name = task.get('name')
-                            tp = task.get('kwargs', {}).get('tp')
-                            if name == self.task_name and tp == self.tp:
-                                execution_ids.append(_id)
-
-                snapshots = AutomationExecution.objects.filter(id__in=execution_ids).values_list('snapshot', flat=True)
-
-                asset_ids = {asset for i in snapshots for asset in i.get('assets', [])}
-                account_ids = {account for i in snapshots for account in i.get('accounts', [])}
-
-                ongoing_counts = (len(execution_ids), len(asset_ids), len(account_ids))
-                data['total_count_ongoing_change_secret'] = ongoing_counts[0]
-                data['total_count_ongoing_change_secret_assets'] = ongoing_counts[1]
-                data['total_count_ongoing_change_secret_accounts'] = ongoing_counts[2]
-                cache.set(self.ongoing_change_secret_cache_key, ongoing_counts, 60)
-            else:
-                data['total_count_ongoing_change_secret'] = ongoing_counts[0]
-                data['total_count_ongoing_change_secret_assets'] = ongoing_counts[1]
-                data['total_count_ongoing_change_secret_accounts'] = ongoing_counts[2]
-
-        return JsonResponse(data, status=200)

apps/accounts/api/automations/check_account.py

@@ -1,162 +0,0 @@
-# -*- coding: utf-8 -*-
-#
-from django.db.models import Q, Count
-from django.http import HttpResponse
-from django.shortcuts import get_object_or_404
-from rest_framework.decorators import action
-from rest_framework.exceptions import MethodNotAllowed
-from rest_framework.response import Response
-
-from accounts import serializers
-from accounts.const import AutomationTypes
-from accounts.models import (
-    CheckAccountAutomation,
-    AccountRisk,
-    RiskChoice,
-    CheckAccountEngine,
-    AutomationExecution,
-)
-from assets.models import Asset
-from common.api import JMSModelViewSet
-from common.permissions import IsValidLicense
-from common.utils import many_get
-from orgs.mixins.api import OrgBulkModelViewSet
-from rbac.permissions import RBACPermission
-from .base import AutomationExecutionViewSet
-from ...filters import NodeFilterBackend
-from ...risk_handlers import RiskHandler
-
-__all__ = [
-    "CheckAccountAutomationViewSet",
-    "CheckAccountExecutionViewSet",
-    "AccountRiskViewSet",
-    "CheckAccountEngineViewSet",
-]
-
-
-class CheckAccountAutomationViewSet(OrgBulkModelViewSet):
-    model = CheckAccountAutomation
-    filterset_fields = ("name",)
-    search_fields = filterset_fields
-    permission_classes = [RBACPermission, IsValidLicense]
-    serializer_class = serializers.CheckAccountAutomationSerializer
-
-
-class CheckAccountExecutionViewSet(AutomationExecutionViewSet):
-    rbac_perms = (
-        ("list", "accounts.view_checkaccountexecution"),
-        ("retrieve", "accounts.view_checkaccountsexecution"),
-        ("create", "accounts.add_checkaccountexecution"),
-        ("adhoc", "accounts.add_checkaccountexecution"),
-        ("report", "accounts.view_checkaccountsexecution"),
-    )
-    ordering = ("-date_created",)
-    tp = AutomationTypes.check_account
-    permission_classes = [RBACPermission, IsValidLicense]
-
-    def get_queryset(self):
-        queryset = super().get_queryset()
-        queryset = queryset.filter(type=self.tp)
-        return queryset
-
-    @action(methods=["get"], detail=False, url_path="adhoc")
-    def adhoc(self, request, *args, **kwargs):
-        asset_id = request.query_params.get("asset_id")
-        if not asset_id:
-            return Response(status=400, data={"asset_id": "This field is required."})
-
-        asset = get_object_or_404(Asset, pk=asset_id)
-        name = "Check asset risk: {}".format(asset.name)
-        execution = AutomationExecution()
-        execution.snapshot = {
-            "assets": [asset_id],
-            "nodes": [],
-            "type": AutomationTypes.check_account,
-            "engines": "__all__",
-            "name": name,
-        }
-        execution.save()
-        execution.start()
-        report = execution.manager.gen_report()
-        return HttpResponse(report)
-
-
-class AccountRiskViewSet(OrgBulkModelViewSet):
-    model = AccountRisk
-    search_fields = ["username", "asset__name"]
-    filterset_fields = ("risk", "status", "asset_id")
-    extra_filter_backends = [NodeFilterBackend]
-    permission_classes = [RBACPermission, IsValidLicense]
-    serializer_classes = {
-        "default": serializers.AccountRiskSerializer,
-        "assets": serializers.AssetRiskSerializer,
-        "handle": serializers.HandleRiskSerializer,
-    }
-    ordering_fields = ("asset", "risk", "status", "username", "date_created")
-    ordering = ("status", "asset", "date_created")
-    rbac_perms = {
-        "sync_accounts": "assets.add_accountrisk",
-        "assets": "accounts.view_accountrisk",
-        "handle": "accounts.change_accountrisk",
-    }
-
-    def update(self, request, *args, **kwargs):
-        raise MethodNotAllowed("PUT")
-
-    def create(self, request, *args, **kwargs):
-        raise MethodNotAllowed("POST")
-
-    @action(methods=["get"], detail=False, url_path="assets")
-    def assets(self, request, *args, **kwargs):
-        annotations = {
-            f"{risk[0]}_count": Count("id", filter=Q(risk=risk[0]))
-            for risk in RiskChoice.choices
-        }
-        queryset = (
-            AccountRisk.objects.select_related(
-                "asset", "asset__platform"
-            )  # 使用 select_related 来优化 asset 和 asset__platform 的查询
-            .values(
-                "asset__id", "asset__name", "asset__address", "asset__platform__name"
-            )  # 添加需要的字段
-            .annotate(risk_total=Count("id"))  # 计算风险总数
-            .annotate(**annotations)  # 使用上面定义的 annotations 进行计数
-        )
-        return self.get_paginated_response_from_queryset(queryset)
-
-    @action(methods=["post"], detail=False, url_path="handle")
-    def handle(self, request, *args, **kwargs):
-        s = self.get_serializer(data=request.data)
-        s.is_valid(raise_exception=True)
-
-        asset, username, act, risk = many_get(
-            s.validated_data, ("asset", "username", "action", "risk")
-        )
-        handler = RiskHandler(asset=asset, username=username, request=self.request)
-
-        try:
-            risk = handler.handle(act, risk)
-            s = serializers.AccountRiskSerializer(instance=risk)
-            return Response(data=s.data)
-        except Exception as e:
-            return Response(status=400, data=str(e))
-
-
-class CheckAccountEngineViewSet(JMSModelViewSet):
-    search_fields = ("name",)
-    serializer_class = serializers.CheckAccountEngineSerializer
-    permission_classes = [RBACPermission, IsValidLicense]
-    perm_model = CheckAccountEngine
-    http_method_names = ['get', 'options']
-
-    def get_queryset(self):
-        return CheckAccountEngine.get_default_engines()
-
-    def filter_queryset(self, queryset: list):
-        search = self.request.GET.get('search')
-        if search is not None:
-            queryset = [
-                item for item in queryset
-                if search in item['name']
-            ]
-        return queryset

apps/accounts/api/automations/gather_account.py

@@ -1,131 +0,0 @@
-# -*- coding: utf-8 -*-
-#
-from django.http import HttpResponse
-from django.shortcuts import get_object_or_404
-from django.views.decorators.clickjacking import xframe_options_sameorigin
-from rest_framework import status
-from rest_framework.decorators import action
-from rest_framework.response import Response
-
-from accounts import serializers
-from accounts.const import AutomationTypes
-from accounts.filters import GatheredAccountFilterSet, NodeFilterBackend
-from accounts.models import GatherAccountsAutomation, AutomationExecution, Account
-from accounts.models import GatheredAccount
-from assets.models import Asset
-from common.const import ConfirmOrIgnore
-from common.utils.http import is_true
-from orgs.mixins.api import OrgBulkModelViewSet
-from .base import AutomationExecutionViewSet
-
-__all__ = [
-    "DiscoverAccountsAutomationViewSet",
-    "DiscoverAccountsExecutionViewSet",
-    "GatheredAccountViewSet",
-]
-
-from ...risk_handlers import RiskHandler
-
-
-class DiscoverAccountsAutomationViewSet(OrgBulkModelViewSet):
-    model = GatherAccountsAutomation
-    filterset_fields = ("name",)
-    search_fields = filterset_fields
-    serializer_class = serializers.DiscoverAccountAutomationSerializer
-
-
-class DiscoverAccountsExecutionViewSet(AutomationExecutionViewSet):
-    rbac_perms = (
-        ("list", "accounts.view_gatheraccountsexecution"),
-        ("retrieve", "accounts.view_gatheraccountsexecution"),
-        ("create", "accounts.add_gatheraccountsexecution"),
-        ("adhoc", "accounts.add_gatheraccountsexecution"),
-        ("report", "accounts.view_gatheraccountsexecution"),
-    )
-
-    tp = AutomationTypes.gather_accounts
-
-    def get_queryset(self):
-        queryset = super().get_queryset()
-        queryset = queryset.filter(type=self.tp)
-        return queryset
-
-    @xframe_options_sameorigin
-    @action(methods=["get"], detail=False, url_path="adhoc")
-    def adhoc(self, request, *args, **kwargs):
-        asset_id = request.query_params.get("asset_id")
-        if not asset_id:
-            return Response(status=400, data={"asset_id": "This field is required."})
-
-        asset = get_object_or_404(Asset, pk=asset_id)
-        execution = AutomationExecution()
-        execution.snapshot = {
-            "assets": [asset_id],
-            "nodes": [],
-            "type": "gather_accounts",
-            "is_sync_account": False,
-            "check_risk": True,
-            "name": "Adhoc gather accounts: {}".format(asset.name),
-        }
-        execution.save()
-        execution.start()
-        report = execution.manager.gen_report()
-        return HttpResponse(report)
-
-
-class GatheredAccountViewSet(OrgBulkModelViewSet):
-    model = GatheredAccount
-    search_fields = ("username",)
-    filterset_class = GatheredAccountFilterSet
-    extra_filter_backends = [NodeFilterBackend]
-    ordering = ("status",)
-    serializer_classes = {
-        "default": serializers.DiscoverAccountSerializer,
-        "status": serializers.DiscoverAccountActionSerializer,
-        "details": serializers.DiscoverAccountDetailsSerializer
-    }
-    rbac_perms = {
-        "status": "assets.change_gatheredaccount",
-        "details": "assets.view_gatheredaccount"
-    }
-
-    @action(methods=["put"], detail=False, url_path="status")
-    def status(self, request, *args, **kwargs):
-        serializer = self.get_serializer(data=request.data)
-        if not serializer.is_valid():
-            return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
-        validated_data = serializer.validated_data
-        ids = validated_data.get('ids', [])
-        new_status = validated_data.get('status')
-        updated_instances = GatheredAccount.objects.filter(id__in=ids).select_related('asset')
-        if new_status == ConfirmOrIgnore.confirmed:
-            GatheredAccount.sync_accounts(updated_instances)
-            updated_instances.update(present=True)
-        updated_instances.update(status=new_status)
-        return Response(status=status.HTTP_200_OK)
-
-    def perform_destroy(self, instance):
-        request = self.request
-        params = request.query_params
-        is_delete_remote = params.get("is_delete_remote")
-        is_delete_account = params.get("is_delete_account")
-        asset_id = params.get("asset")
-        username = params.get("username")
-        if is_true(is_delete_remote):
-            self._delete_remote(asset_id, username)
-        if is_true(is_delete_account):
-            account = get_object_or_404(Account, username=username, asset_id=asset_id)
-            account.delete()
-        super().perform_destroy(instance)
-
-    def _delete_remote(self, asset_id, username):
-        asset = get_object_or_404(Asset, pk=asset_id)
-        handler = RiskHandler(asset, username, request=self.request)
-        handler.handle_delete_remote()
-
-    @action(methods=["get"], detail=True, url_path="details")
-    def details(self, request, *args, **kwargs):
-        pk = kwargs.get('pk')
-        account = get_object_or_404(GatheredAccount, pk=pk)
-        serializer = self.get_serializer(account.detail)
-        return Response(data=serializer.data)

apps/accounts/api/automations/gather_accounts.py

@@ -0,0 +1,59 @@
+# -*- coding: utf-8 -*-
+#
+from rest_framework import status
+from rest_framework.decorators import action
+from rest_framework.response import Response
+
+from accounts import serializers
+from accounts.const import AutomationTypes
+from accounts.filters import GatheredAccountFilterSet
+from accounts.models import GatherAccountsAutomation
+from accounts.models import GatheredAccount
+from orgs.mixins.api import OrgBulkModelViewSet
+from .base import AutomationExecutionViewSet
+
+__all__ = [
+    'GatherAccountsAutomationViewSet', 'GatherAccountsExecutionViewSet',
+    'GatheredAccountViewSet'
+]
+
+
+class GatherAccountsAutomationViewSet(OrgBulkModelViewSet):
+    model = GatherAccountsAutomation
+    filterset_fields = ('name',)
+    search_fields = filterset_fields
+    serializer_class = serializers.GatherAccountAutomationSerializer
+
+
+class GatherAccountsExecutionViewSet(AutomationExecutionViewSet):
+    rbac_perms = (
+        ("list", "accounts.view_gatheraccountsexecution"),
+        ("retrieve", "accounts.view_gatheraccountsexecution"),
+        ("create", "accounts.add_gatheraccountsexecution"),
+    )
+
+    tp = AutomationTypes.gather_accounts
+
+    def get_queryset(self):
+        queryset = super().get_queryset()
+        queryset = queryset.filter(automation__type=self.tp)
+        return queryset
+
+
+class GatheredAccountViewSet(OrgBulkModelViewSet):
+    model = GatheredAccount
+    search_fields = ('username',)
+    filterset_class = GatheredAccountFilterSet
+    serializer_classes = {
+        'default': serializers.GatheredAccountSerializer,
+    }
+    rbac_perms = {
+        'sync_accounts': 'assets.add_gatheredaccount',
+    }
+
+    @action(methods=['post'], detail=False, url_path='sync-accounts')
+    def sync_accounts(self, request, *args, **kwargs):
+        gathered_account_ids = request.data.get('gathered_account_ids')
+        gathered_accounts = self.model.objects.filter(id__in=gathered_account_ids)
+        self.model.sync_accounts(gathered_accounts)
+        return Response(status=status.HTTP_201_CREATED)

apps/accounts/api/automations/push_account.py

@@ -1,16 +1,15 @@
 # -*- coding: utf-8 -*-
 #
-from rest_framework import mixins
-
 from accounts import serializers
 from accounts.const import AutomationTypes
-from accounts.filters import PushAccountRecordFilterSet
-from accounts.models import PushAccountAutomation, PushSecretRecord
-from orgs.mixins.api import OrgBulkModelViewSet, OrgGenericViewSet
+from accounts.models import PushAccountAutomation, ChangeSecretRecord
+from orgs.mixins.api import OrgBulkModelViewSet
+
 from .base import (
     AutomationAssetsListApi, AutomationRemoveAssetApi, AutomationAddAssetApi,
     AutomationNodeAddRemoveApi, AutomationExecutionViewSet
 )
+from .change_secret import ChangeSecretRecordViewSet
 
 __all__ = [
     'PushAccountAutomationViewSet', 'PushAccountAssetsListApi', 'PushAccountRemoveAssetApi',
@@ -31,28 +30,24 @@ class PushAccountExecutionViewSet(AutomationExecutionViewSet):
         ("list", "accounts.view_pushaccountexecution"),
         ("retrieve", "accounts.view_pushaccountexecution"),
         ("create", "accounts.add_pushaccountexecution"),
-        ("report", "accounts.view_pushaccountexecution"),
     )
 
     tp = AutomationTypes.push_account
 
     def get_queryset(self):
         queryset = super().get_queryset()
-        queryset = queryset.filter(type=self.tp)
+        queryset = queryset.filter(automation__type=self.tp)
         return queryset
 
 
-class PushAccountRecordViewSet(mixins.ListModelMixin, OrgGenericViewSet):
-    filterset_class = PushAccountRecordFilterSet
-    search_fields = ('asset__address', 'account__username')
-    ordering_fields = ('date_finished',)
+class PushAccountRecordViewSet(ChangeSecretRecordViewSet):
+    serializer_class = serializers.ChangeSecretRecordSerializer
     tp = AutomationTypes.push_account
-    serializer_classes = {
-        'default': serializers.PushSecretRecordSerializer,
-    }
 
     def get_queryset(self):
-        return PushSecretRecord.get_valid_records()
+        return ChangeSecretRecord.objects.filter(
+            execution__automation__type=AutomationTypes.push_account
+        )
 
 
 class PushAccountAssetsListApi(AutomationAssetsListApi):

apps/accounts/automations/backup_account/handlers.py

@@ -3,17 +3,15 @@ import time
 from collections import defaultdict, OrderedDict
 
 from django.conf import settings
-from django.db.models import F
 from django.utils.translation import gettext_lazy as _
 from rest_framework import serializers
 from xlsxwriter import Workbook
 
 from accounts.const import AccountBackupType
-from accounts.models import BackupAccountAutomation, Account
+from accounts.models.automations.backup_account import AccountBackupAutomation
 from accounts.notifications import AccountBackupExecutionTaskMsg, AccountBackupByObjStorageExecutionTaskMsg
 from accounts.serializers import AccountSecretSerializer
 from assets.const import AllTypes
-from common.const import Status
 from common.utils.file import encrypt_and_compress_zip_file, zip_files
 from common.utils.timezone import local_now_filename, local_now_display
 from terminal.models.component.storage import ReplayStorage
@@ -22,7 +20,6 @@ from users.models import User
 PATH = os.path.join(os.path.dirname(settings.BASE_DIR), 'tmp')
 split_help_text = _('The account key will be split into two parts and sent')
 
-
 class RecipientsNotFound(Exception):
     pass
 
@@ -36,26 +33,17 @@ class BaseAccountHandler:
             if isinstance(v, OrderedDict):
                 cls.unpack_data(v, data)
             else:
-                if isinstance(v, dict):
-                    v = v.get('label')
-                elif v is None:
-                    v = ''
                 data[k] = v
         return data
 
     @classmethod
     def get_header_fields(cls, serializer: serializers.Serializer):
         try:
-            exclude_backup_fields = getattr(serializer, 'Meta').exclude_backup_fields
+            backup_fields = getattr(serializer, 'Meta').fields_backup
         except AttributeError:
-            exclude_backup_fields = []
-        backup_fields = serializer.fields.keys()
-
+            backup_fields = serializer.fields.keys()
         header_fields = {}
         for field in backup_fields:
-            if field in exclude_backup_fields:
-                continue
-
             v = serializer.fields[field]
             if isinstance(v, serializers.Serializer):
                 _fields = cls.get_header_fields(v)
@@ -85,9 +73,9 @@ class BaseAccountHandler:
 
 class AssetAccountHandler(BaseAccountHandler):
     @staticmethod
-    def get_filename(name):
+    def get_filename(plan_name):
         filename = os.path.join(
-            PATH, f'{name}-{local_now_filename()}-{time.time()}.xlsx'
+            PATH, f'{plan_name}-{local_now_filename()}-{time.time()}.xlsx'
         )
         return filename
 
@@ -129,41 +117,32 @@ class AssetAccountHandler(BaseAccountHandler):
             cls.handler_secret(data, section)
             data_map.update(cls.add_rows(data, header_fields, sheet_name))
         number_of_backup_accounts = _('Number of backup accounts')
-        print('\033[33m- {}: {}\033[0m'.format(number_of_backup_accounts, accounts.count()))
+        print('\n\033[33m- {}: {}\033[0m'.format(number_of_backup_accounts, accounts.count()))
         return data_map
 
 
 class AccountBackupHandler:
-    def __init__(self, manager, execution):
-        self.manager = manager
+    def __init__(self, execution):
         self.execution = execution
-        self.name = self.execution.snapshot.get('name', '-')
-
-    def get_accounts(self):
-        # TODO 可以优化一下查询 在账号上做 category 的缓存 避免数据量大时连表操作
-        types = self.execution.snapshot.get('types', [])
-        self.manager.summary['total_types'] = len(types)
-        qs = Account.objects.filter(
-            asset__platform__type__in=types
-        ).annotate(type=F('asset__platform__type'))
-        return qs
+        self.plan_name = self.execution.plan.name
+        self.is_frozen = False  # 任务状态冻结标志
 
     def create_excel(self, section='complete'):
-        hint = _('Generating asset related backup information files')
+        hint = _('Generating asset or application related backup information files')
         print(
+            '\n'
             f'\033[32m>>> {hint}\033[0m'
             ''
         )
-
+        # Print task start date
         time_start = time.time()
         files = []
-        accounts = self.get_accounts()
-        self.manager.summary['total_accounts'] = accounts.count()
+        accounts = self.execution.backup_accounts
         data_map = AssetAccountHandler.create_data_map(accounts, section)
         if not data_map:
             return files
 
-        filename = AssetAccountHandler.get_filename(self.name)
+        filename = AssetAccountHandler.get_filename(self.plan_name)
 
         wb = Workbook(filename)
         for sheet, data in data_map.items():
@@ -174,7 +153,7 @@ class AccountBackupHandler:
         wb.close()
         files.append(filename)
         timedelta = round((time.time() - time_start), 2)
-        time_cost = _('Duration')
+        time_cost = _('Time cost')
         file_created = _('Backup file creation completed')
         print('{}: {} {}s'.format(file_created, time_cost, timedelta))
         return files
@@ -183,21 +162,22 @@ class AccountBackupHandler:
         if not files:
             return
         recipients = User.objects.filter(id__in=list(recipients))
-        msg = _("Start sending backup emails")
         print(
-            f'\033[32m>>> {msg}\033[0m'
+            '\n'
+            f'\033[32m>>> {_("Start sending backup emails")}\033[0m'
             ''
         )
-        name = self.name
+        plan_name = self.plan_name
         for user in recipients:
             if not user.secret_key:
                 attachment_list = []
             else:
-                attachment = os.path.join(PATH, f'{name}-{local_now_filename()}-{time.time()}.zip')
+                attachment = os.path.join(PATH, f'{plan_name}-{local_now_filename()}-{time.time()}.zip')
                 encrypt_and_compress_zip_file(attachment, user.secret_key, files)
-                attachment_list = [attachment]
-            AccountBackupExecutionTaskMsg(name, user).publish(attachment_list)
-
+                attachment_list = [attachment, ]
+            AccountBackupExecutionTaskMsg(plan_name, user).publish(attachment_list)
+            email_sent_to = _('Email sent to')
+            print('{} {}({})'.format(email_sent_to, user, user.email))
         for file in files:
             os.remove(file)
 
@@ -206,41 +186,63 @@ class AccountBackupHandler:
             return
         recipients = ReplayStorage.objects.filter(id__in=list(recipients))
         print(
+            '\n'
             '\033[32m>>> 📃 ---> sftp \033[0m'
             ''
         )
-        name = self.name
+        plan_name = self.plan_name
         encrypt_file = _('Encrypting files using encryption password')
         for rec in recipients:
-            attachment = os.path.join(PATH, f'{name}-{local_now_filename()}-{time.time()}.zip')
+            attachment = os.path.join(PATH, f'{plan_name}-{local_now_filename()}-{time.time()}.zip')
             if password:
                 print(f'\033[32m>>> {encrypt_file}\033[0m')
                 encrypt_and_compress_zip_file(attachment, password, files)
             else:
                 zip_files(attachment, files)
             attachment_list = attachment
-            AccountBackupByObjStorageExecutionTaskMsg(name, rec).publish(attachment_list)
+            AccountBackupByObjStorageExecutionTaskMsg(plan_name, rec).publish(attachment_list)
             file_sent_to = _('The backup file will be sent to')
             print('{}: {}({})'.format(file_sent_to, rec.name, rec.id))
         for file in files:
             os.remove(file)
 
+    def step_perform_task_update(self, is_success, reason):
+        self.execution.reason = reason[:1024]
+        self.execution.is_success = is_success
+        self.execution.save()
+        finish = _('Finish')
+        print(f'\n{finish}\n')
+
+    @staticmethod
+    def step_finished(is_success):
+        if is_success:
+            print(_('Success'))
+        else:
+            print(_('Failed'))
+
     def _run(self):
+        is_success = False
+        error = '-'
         try:
-            backup_type = self.execution.snapshot.get('backup_type', AccountBackupType.email)
-            if backup_type == AccountBackupType.email:
+            backup_type = self.execution.snapshot.get('backup_type', AccountBackupType.email.value)
+            if backup_type == AccountBackupType.email.value:
                 self.backup_by_email()
-            elif backup_type == AccountBackupType.object_storage:
+            elif backup_type == AccountBackupType.object_storage.value:
                 self.backup_by_obj_storage()
         except Exception as e:
+            self.is_frozen = True
+            print(e)
             error = str(e)
-            print(f'\033[31m>>> {error}\033[0m')
-            self.execution.status = Status.error
-            self.execution.summary['error'] = error
+        else:
+            is_success = True
+        finally:
+            reason = error
+            self.step_perform_task_update(is_success, reason)
+            self.step_finished(is_success)
 
     def backup_by_obj_storage(self):
         object_id = self.execution.snapshot.get('id')
-        zip_encrypt_password = BackupAccountAutomation.objects.get(id=object_id).zip_encrypt_password
+        zip_encrypt_password = AccountBackupAutomation.objects.get(id=object_id).zip_encrypt_password
         obj_recipients_part_one = self.execution.snapshot.get('obj_recipients_part_one', [])
         obj_recipients_part_two = self.execution.snapshot.get('obj_recipients_part_two', [])
         no_assigned_sftp_server = _('The backup task has no assigned sftp server')
@@ -264,6 +266,7 @@ class AccountBackupHandler:
             self.send_backup_obj_storage(files, recipients, zip_encrypt_password)
 
     def backup_by_email(self):
+
         warn_text = _('The backup task has no assigned recipient')
         recipients_part_one = self.execution.snapshot.get('recipients_part_one', [])
         recipients_part_two = self.execution.snapshot.get('recipients_part_two', [])
@@ -273,7 +276,7 @@ class AccountBackupHandler:
                 f'\033[31m>>> {warn_text}\033[0m'
                 ''
             )
-            return
+            raise RecipientsNotFound('Not Found Recipients')
         if recipients_part_one and recipients_part_two:
             print(f'\033[32m>>> {split_help_text}\033[0m')
             files = self.create_excel(section='front')
@@ -287,5 +290,18 @@ class AccountBackupHandler:
             self.send_backup_mail(files, recipients)
 
     def run(self):
-        print('{}: {}'.format(_('Plan start'), local_now_display()))
-        self._run()
+        plan_start = _('Plan start')
+        plan_end = _('Plan end')
+        time_cost = _('Time cost')
+        error = _('An exception occurred during task execution')
+        print('{}: {}'.format(plan_start, local_now_display()))
+        time_start = time.time()
+        try:
+            self._run()
+        except Exception as e:
+            print(error)
+            print(e)
+        finally:
+            print('\n{}: {}'.format(plan_end, local_now_display()))
+            timedelta = round((time.time() - time_start), 2)
+            print('{}: {}s'.format(time_cost, timedelta))

apps/accounts/automations/backup_account/manager.py

@@ -1,30 +1,48 @@
 # -*- coding: utf-8 -*-
 #
+import time
 
+from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 
-from assets.automations.base.manager import BaseManager
 from common.utils.timezone import local_now_display
 from .handlers import AccountBackupHandler
 
 
-class AccountBackupManager(BaseManager):
+class AccountBackupManager:
+    def __init__(self, execution):
+        self.execution = execution
+        self.date_start = timezone.now()
+        self.time_start = time.time()
+        self.date_end = None
+        self.time_end = None
+        self.timedelta = 0
+
     def do_run(self):
         execution = self.execution
         account_backup_execution_being_executed = _('The account backup plan is being executed')
-        print(f'\033[33m# {account_backup_execution_being_executed}\033[0m')
-        handler = AccountBackupHandler(self, execution)
+        print(f'\n\033[33m# {account_backup_execution_being_executed}\033[0m')
+        handler = AccountBackupHandler(execution)
         handler.run()
 
-    def send_report_if_need(self):
-        pass
+    def pre_run(self):
+        self.execution.date_start = self.date_start
+        self.execution.save()
+
+    def post_run(self):
+        self.time_end = time.time()
+        self.date_end = timezone.now()
 
-    def print_summary(self):
         print('\n\n' + '-' * 80)
         plan_execution_end = _('Plan execution end')
         print('{} {}\n'.format(plan_execution_end, local_now_display()))
-        time_cost = _('Duration')
-        print('{}: {}s'.format(time_cost, self.duration))
+        self.timedelta = self.time_end - self.time_start
+        time_cost = _('Time cost')
+        print('{}: {}s'.format(time_cost, self.timedelta))
+        self.execution.timedelta = self.timedelta
+        self.execution.save()
 
-    def get_report_template(self):
-        return "accounts/backup_account_report.html"
+    def run(self):
+        self.pre_run()
+        self.do_run()
+        self.post_run()

apps/accounts/automations/base/manager.py

@@ -1,203 +1,12 @@
-from copy import deepcopy
-
-from django.conf import settings
-from django.utils import timezone
-from django.utils.translation import gettext_lazy as _
-
 from accounts.automations.methods import platform_automation_methods
-from accounts.const import SSHKeyStrategy, SecretStrategy, SecretType, ChangeSecretRecordStatusChoice
-from accounts.models import BaseAccountQuerySet
-from accounts.utils import SecretGenerator
 from assets.automations.base.manager import BasePlaybookManager
-from assets.const import HostTypes
-from common.db.utils import safe_db_connection
 from common.utils import get_logger
 
 logger = get_logger(__name__)
 
 
 class AccountBasePlaybookManager(BasePlaybookManager):
-    template_path = ''
 
     @property
     def platform_automation_methods(self):
         return platform_automation_methods
-
-
-class BaseChangeSecretPushManager(AccountBasePlaybookManager):
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self.secret_type = self.execution.snapshot.get('secret_type')
-        self.secret_strategy = self.execution.snapshot.get(
-            'secret_strategy', SecretStrategy.custom
-        )
-        self.ssh_key_change_strategy = self.execution.snapshot.get(
-            'ssh_key_change_strategy', SSHKeyStrategy.set_jms
-        )
-        self.account_ids = self.execution.snapshot['accounts']
-        self.record_map = self.execution.snapshot.get('record_map', {})  # 这个是某个失败的记录重试
-        self.name_recorder_mapper = {}  # 做个映射，方便后面处理
-
-    def gen_account_inventory(self, account, asset, h, path_dir):
-        raise NotImplementedError
-
-    def get_ssh_params(self, secret, secret_type):
-        kwargs = {}
-        if secret_type != SecretType.SSH_KEY:
-            return kwargs
-        kwargs['strategy'] = self.ssh_key_change_strategy
-        kwargs['exclusive'] = 'yes' if kwargs['strategy'] == SSHKeyStrategy.set else 'no'
-
-        if kwargs['strategy'] == SSHKeyStrategy.set_jms:
-            kwargs['regexp'] = '.*{}$'.format(secret.split()[2].strip())
-        return kwargs
-
-    def get_secret(self, account):
-        if self.secret_strategy == SecretStrategy.custom:
-            new_secret = self.execution.snapshot['secret']
-        else:
-            generator = SecretGenerator(
-                self.secret_strategy, self.secret_type,
-                self.execution.snapshot.get('password_rules')
-            )
-            new_secret = generator.get_secret()
-        return new_secret
-
-    def get_accounts(self, privilege_account) -> BaseAccountQuerySet | None:
-        if not privilege_account:
-            print('Not privilege account')
-            return
-
-        asset = privilege_account.asset
-        accounts = asset.all_accounts.all()
-        accounts = accounts.filter(id__in=self.account_ids, secret_reset=True)
-
-        if self.secret_type:
-            accounts = accounts.filter(secret_type=self.secret_type)
-
-        if settings.CHANGE_AUTH_PLAN_SECURE_MODE_ENABLED:
-            accounts = accounts.filter(privileged=False).exclude(
-                username__in=['root', 'administrator', privilege_account.username]
-            )
-        return accounts
-
-    def handle_ssh_secret(self, secret_type, new_secret, path_dir):
-        private_key_path = None
-        if secret_type == SecretType.SSH_KEY:
-            private_key_path = self.generate_private_key_path(new_secret, path_dir)
-            new_secret = self.generate_public_key(new_secret)
-        return new_secret, private_key_path
-
-    def gen_inventory(self, h, account, new_secret, private_key_path, asset):
-        secret_type = account.secret_type
-        h['ssh_params'].update(self.get_ssh_params(new_secret, secret_type))
-        h['account'] = {
-            'name': account.name,
-            'username': account.username,
-            'full_username': account.full_username,
-            'secret_type': secret_type,
-            'secret': account.escape_jinja2_syntax(new_secret),
-            'private_key_path': private_key_path,
-            'become': account.get_ansible_become_auth(),
-        }
-        if asset.platform.type == 'oracle':
-            h['account']['mode'] = 'sysdba' if account.privileged else None
-        return h
-
-    def host_callback(self, host, asset=None, account=None, automation=None, path_dir=None, **kwargs):
-        host = super().host_callback(
-            host, asset=asset, account=account, automation=automation,
-            path_dir=path_dir, **kwargs
-        )
-        if host.get('error'):
-            return host
-
-        host['check_conn_after_change'] = self.execution.snapshot.get('check_conn_after_change', True)
-        host['ssh_params'] = {}
-
-        accounts = self.get_accounts(account)
-        error_msg = _("No pending accounts found")
-        if not accounts:
-            print(f'{asset}: {error_msg}')
-            return []
-
-        if asset.type == HostTypes.WINDOWS:
-            accounts = accounts.filter(secret_type=SecretType.PASSWORD)
-
-        inventory_hosts = []
-        if asset.type == HostTypes.WINDOWS and self.secret_type == SecretType.SSH_KEY:
-            print(f'Windows {asset} does not support ssh key push')
-            return inventory_hosts
-
-        for account in accounts:
-            h = deepcopy(host)
-            h['name'] += '(' + account.username + ')'  # To distinguish different accounts
-            try:
-                h = self.gen_account_inventory(account, asset, h, path_dir)
-            except Exception as e:
-                h['error'] = str(e)
-            inventory_hosts.append(h)
-
-        return inventory_hosts
-
-    @staticmethod
-    def save_record(recorder):
-        recorder.save(update_fields=['error', 'status', 'date_finished'])
-
-    def on_host_success(self, host, result):
-        recorder = self.name_recorder_mapper.get(host)
-        if not recorder:
-            return
-        recorder.status = ChangeSecretRecordStatusChoice.success.value
-        recorder.date_finished = timezone.now()
-
-        account = recorder.account
-        if not account:
-            print("Account not found, deleted ?")
-            return
-
-        account.secret = getattr(recorder, 'new_secret', account.secret)
-        account.date_updated = timezone.now()
-        account.date_change_secret = timezone.now()
-        account.change_secret_status = ChangeSecretRecordStatusChoice.success
-
-        self.summary['ok_accounts'] += 1
-        self.result['ok_accounts'].append(
-            {
-                "asset": str(account.asset),
-                "username": account.username,
-            }
-        )
-        super().on_host_success(host, result)
-
-        with safe_db_connection():
-            account.save(update_fields=['secret', 'date_updated', 'date_change_secret', 'change_secret_status'])
-            self.save_record(recorder)
-
-    def on_host_error(self, host, error, result):
-        recorder = self.name_recorder_mapper.get(host)
-        if not recorder:
-            return
-        recorder.status = ChangeSecretRecordStatusChoice.failed.value
-        recorder.date_finished = timezone.now()
-        recorder.error = error
-        account = recorder.account
-        if not account:
-            print("Account not found, deleted ?")
-            return
-        account.date_updated = timezone.now()
-        account.date_change_secret = timezone.now()
-        account.change_secret_status = ChangeSecretRecordStatusChoice.failed
-
-        self.summary['fail_accounts'] += 1
-        self.result['fail_accounts'].append(
-            {
-                "asset": str(recorder.asset),
-                "username": recorder.account.username,
-            }
-        )
-        super().on_host_error(host, error, result)
-
-        with safe_db_connection():
-            account.save(update_fields=['change_secret_status', 'date_change_secret', 'date_updated'])
-            self.save_record(recorder)

apps/accounts/automations/change_secret/custom/ssh/main.yml

@@ -20,7 +20,6 @@
         become_private_key_path: "{{ jms_custom_become_private_key_path | default(None) }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
         gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
-        recv_timeout: "{{ params.recv_timeout | default(30) }}"
       register: ping_info
       delegate_to: localhost
 
@@ -40,12 +39,9 @@
         name: "{{ account.username }}"
         password: "{{ account.secret }}"
         commands: "{{ params.commands }}"
-        answers: "{{ params.answers }}"
-        recv_timeout: "{{ params.recv_timeout | default(30) }}"
-        delay_time: "{{ params.delay_time | default(2) }}"
-        prompt: "{{ params.prompt | default('.*') }}"
+        first_conn_delay_time: "{{ first_conn_delay_time | default(0.5) }}"
       ignore_errors: true
-      when: ping_info is succeeded and check_conn_after_change
+      when: ping_info is succeeded
       register: change_info
       delegate_to: localhost
 
@@ -62,6 +58,4 @@
         become_private_key_path: "{{ account.become.ansible_ssh_private_key_file | default(None) }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
         gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
-        recv_timeout: "{{ params.recv_timeout | default(30) }}"
       delegate_to: localhost
-      when: check_conn_after_change

apps/accounts/automations/change_secret/custom/ssh/manifest.yml

@@ -10,30 +10,10 @@ protocol: ssh
 priority: 50
 params:
   - name: commands
-    type: text
+    type: list
     label: "{{ 'Params commands label' | trans }}"
-    default: ''
+    default: [ '' ]
     help_text: "{{ 'Params commands help text' | trans }}"
-  - name: recv_timeout
-    type: int
-    label: "{{ 'Params recv_timeout label' | trans }}"
-    default: 30
-    help_text: "{{ 'Params recv_timeout help text' | trans }}"
-  - name: delay_time
-    type: int
-    label: "{{ 'Params delay_time label' | trans }}"
-    default: 2
-    help_text: "{{ 'Params delay_time help text' | trans }}"
-  - name: prompt
-    type: str
-    label: "{{ 'Params prompt label' | trans }}"
-    default: '.*'
-    help_text: "{{ 'Params prompt help text' | trans }}"
-  - name: answers
-    type: text
-    label: "{{ 'Params answer label' | trans }}"
-    default: '.*'
-    help_text: "{{ 'Params answer help text' | trans }}"
 
 i18n:
   SSH account change secret:
@@ -42,91 +22,11 @@ i18n:
     en: 'Custom password change by SSH command line'
 
   Params commands help text:
-    zh: |
-      请将命令中的指定位置改成特殊符号 <br />
-      1. 改密账号 -> {username} <br />
-      2. 改密密码 -> {password} <br />
-      3. 登录用户密码 -> {login_password} <br />
-      <strong>多条命令使用换行分割，</strong>执行任务时系统会根据特殊符号替换真实数据。<br />
-      比如针对 Cisco 主机进行改密，一般需要配置五条命令：<br />
-      enable <br />
-      {login_password} <br />
-      configure terminal <br />
-      username {username} privilege 0 password {password} <br />
-      end <br />
-    ja: |
-      コマンド内の指定された位置を特殊記号に変更してください。<br />
-      新しいパスワード（アカウント変更） -> {username} <br />
-      新しいパスワード（パスワード変更） -> {password} <br />
-      ログインユーザーパスワード -> {login_password} <br />
-      <strong>複数のコマンドは改行で区切り、</strong>タスクを実行するときにシステムは特殊記号を使用して実際のデータを置き換えます。<br />
-      例えば、Cisco機器のパスワードを変更する場合、一般的には5つのコマンドを設定する必要があります：<br />
-      enable <br />
-      {login_password} <br />
-      configure terminal <br />
-      username {username} privilege 0 password {password} <br />
-      end <br />
-    en: |
-      Please change the specified positions in the command to special symbols. <br /> 
-      Change password account -> {username} <br /> 
-      Change password -> {password} <br /> 
-      Login user password -> {login_password} <br /> 
-      <strong>Multiple commands are separated by new lines,</strong> and when executing tasks, <br /> 
-      the system will replace the special symbols with real data. <br /> 
-      For example, to change the password for a Cisco device, you generally need to configure five commands: <br />
-      enable <br />
-      {login_password} <br />
-      configure terminal <br />
-      username {username} privilege 0 password {password} <br />
-      end <br />
+    zh: '自定义命令中如需包含账号的 账号、密码、SSH 连接的用户密码 字段，<br />请使用 &#123;username&#125;、&#123;password&#125;、&#123;login_password&#125;格式，执行任务时会进行替换 。<br />比如针对 Cisco 主机进行改密，一般需要配置五条命令：<br />1. enable<br />2. &#123;login_password&#125;<br />3. configure terminal<br />4. username &#123;username&#125; privilege 0 password &#123;password&#125; <br />5. end'
+    ja: 'カスタム コマンドに SSH 接続用のアカウント番号、パスワード、ユーザー パスワード フィールドを含める必要がある場合は、<br />&#123;ユーザー名&#125;、&#123;パスワード&#125;、&#123;login_password& を使用してください。 # 125; 形式。タスクの実行時に置き換えられます。 <br />たとえば、Cisco ホストのパスワードを変更するには、通常、次の 5 つのコマンドを設定する必要があります:<br />1.enable<br />2.&#123;login_password&#125;<br />3 .ターミナルの設定<br / >4. ユーザー名 &#123;ユーザー名&#125; 権限 0 パスワード &#123;パスワード&#125; <br />5. 終了'
+    en: 'If the custom command needs to include the account number, password, and user password field for SSH connection,<br />Please use &#123;username&#125;, &#123;password&#125;, &#123;login_password&# 125; format, which will be replaced when executing the task. <br />For example, to change the password of a Cisco host, you generally need to configure five commands:<br />1. enable<br />2. &#123;login_password&#125;<br />3. configure terminal<br / >4. username &#123;username&#125; privilege 0 password &#123;password&#125; <br />5. end'
 
   Params commands label:
     zh: '自定义命令'
     ja: 'カスタムコマンド'
     en: 'Custom command'
-
-  Params recv_timeout label:
-    zh: '超时时间'
-    ja: 'タイムアウト'
-    en: 'Timeout'
-
-  Params recv_timeout help text:
-    zh: '等待命令结果返回的超时时间（秒）'
-    ja: 'コマンドの結果を待つタイムアウト時間（秒）'
-    en: 'The timeout for waiting for the command result to return (Seconds)'
-
-  Params delay_time label:
-    zh: '延迟发送时间'
-    ja: '遅延送信時間'
-    en: 'Delayed send time'
-
-  Params delay_time help text:
-    zh: '每条命令延迟发送的时间间隔（秒）'
-    ja: '各コマンド送信の遅延間隔（秒）'
-    en: 'Time interval for each command delay in sending (Seconds)'
-
-  Params prompt label:
-    zh: '提示符'
-    ja: 'ヒント'
-    en: 'Prompt'
-
-  Params prompt help text:
-    zh: '终端连接后显示的提示符信息（正则表达式）'
-    ja: 'ターミナル接続後に表示されるプロンプト情報（正規表現）'
-    en: 'Prompt information displayed after terminal connection (Regular expression)'
-
-  Params answer label:
-    zh: '命令结果'
-    ja: 'コマンド結果'
-    en: 'Command result'
-
-  Params answer help text:
-    zh: |
-      根据结果匹配度决定是否执行下一条命令，输入框的内容和上方 “自定义命令” 内容按行一一对应（正则表达式）
-    ja: |
-      結果の一致度に基づいて次のコマンドを実行するかどうかを決定します。
-      入力欄の内容は、上の「カスタムコマンド」の内容と行ごとに対応しています（せいきひょうげん）
-    en: |
-      Decide whether to execute the next command based on the result match. 
-      The input content corresponds line by line with the content 
-      of the `Custom command` above. (Regular expression)

apps/accounts/automations/change_secret/database/mongodb/main.yml

@@ -1,7 +1,7 @@
 - hosts: mongodb
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test MongoDB connection
@@ -53,4 +53,3 @@
         ssl_certfile: "{{ jms_asset.secret_info.client_key | default('') }}"
         connection_options:
           - tlsAllowInvalidHostnames: "{{ jms_asset.spec_info.allow_invalid_cert}}"
-      when: check_conn_after_change

apps/accounts/automations/change_secret/database/mysql/main.yml

@@ -1,12 +1,9 @@
 - hosts: mysql
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
     db_name: "{{ jms_asset.spec_info.db_name }}"
     check_ssl: "{{ jms_asset.spec_info.use_ssl and not jms_asset.spec_info.allow_invalid_cert }}"
-    ca_cert: "{{ jms_asset.secret_info.ca_cert | default('') }}"
-    ssl_cert: "{{ jms_asset.secret_info.client_cert | default('') }}"
-    ssl_key: "{{ jms_asset.secret_info.client_key | default('') }}"
 
   tasks:
     - name: Test MySQL connection
@@ -16,9 +13,9 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         filter: version
       register: db_info
 
@@ -33,9 +30,9 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         name: "{{ account.username }}"
         password: "{{ account.secret }}"
         host: "%"
@@ -50,8 +47,7 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         filter: version
-      when: check_conn_after_change

apps/accounts/automations/change_secret/database/oracle/main.yml

@@ -1,7 +1,7 @@
 - hosts: oracle
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test Oracle connection
@@ -40,4 +40,3 @@
         login_port: "{{ jms_asset.port }}"
         login_database: "{{ jms_asset.spec_info.db_name }}"
         mode: "{{ account.mode }}"
-      when: check_conn_after_change

apps/accounts/automations/change_secret/database/postgresql/main.yml

@@ -1,11 +1,7 @@
 - hosts: postgre
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
-    check_ssl: "{{ jms_asset.spec_info.use_ssl }}"
-    ca_cert: "{{ jms_asset.secret_info.ca_cert | default('') }}"
-    ssl_cert: "{{ jms_asset.secret_info.client_cert | default('') }}"
-    ssl_key: "{{ jms_asset.secret_info.client_key | default('') }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test PostgreSQL connection
@@ -15,10 +11,6 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         login_db: "{{ jms_asset.spec_info.db_name }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
       register: result
       failed_when: not result.is_available
 
@@ -36,10 +28,6 @@
         db: "{{ jms_asset.spec_info.db_name }}"
         name: "{{ account.username }}"
         password: "{{ account.secret }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
         role_attr_flags: LOGIN
       ignore_errors: true
       when: result is succeeded
@@ -51,8 +39,3 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         db: "{{ jms_asset.spec_info.db_name }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
-      when: check_conn_after_change

apps/accounts/automations/change_secret/database/sqlserver/main.yml

@@ -1,7 +1,7 @@
 - hosts: sqlserver
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test SQLServer connection
@@ -64,4 +64,3 @@
         name: '{{ jms_asset.spec_info.db_name }}'
         script: |
           SELECT @@version
-      when: check_conn_after_change

apps/accounts/automations/change_secret/host/aix/main.yml

@@ -9,20 +9,55 @@
         database: passwd
         key: "{{ account.username }}"
       register: user_info
-      failed_when: false
-      changed_when: false
+      ignore_errors: yes  # 忽略错误，如果用户不存在时不会导致playbook失败
 
     - name: "Add {{ account.username }} user"
       ansible.builtin.user:
         name: "{{ account.username }}"
-        uid: "{{ params.uid | int if params.uid | length > 0 else omit }}"
-        shell: "{{ params.shell if params.shell | length > 0 else omit }}"
-        home: "{{ params.home if params.home | length > 0 else '/home/' + account.username }}"
-        groups: "{{ params.groups if params.groups | length > 0 else omit }}"
-        append: "{{ true if params.groups | length > 0 else false }}"
+        shell: "{{ params.shell }}"
+        home: "{{ params.home | default('/home/' + account.username, true) }}"
+        groups: "{{ params.groups }}"
         expires: -1
         state: present
-      when: user_info.msg is defined
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} group"
+      ansible.builtin.group:
+        name: "{{ account.username }}"
+        state: present
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} user to group"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        groups: "{{ params.groups }}"
+      when:
+        - user_info.failed
+        - params.groups
+
+    - name: "Change {{ account.username }} password"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        password: "{{ account.secret | password_hash('des') }}"
+        update_password: always
+      ignore_errors: true
+      when: account.secret_type == "password"
+
+    - name: remove jumpserver ssh key
+      ansible.builtin.lineinfile:
+        dest: "{{ ssh_params.dest }}"
+        regexp: "{{ ssh_params.regexp }}"
+        state: absent
+      when:
+        - account.secret_type == "ssh_key"
+        - ssh_params.strategy == "set_jms"
+
+    - name: "Change {{ account.username }} SSH key"
+      ansible.builtin.authorized_key:
+        user: "{{ account.username }}"
+        key: "{{ account.secret }}"
+        exclusive: "{{ ssh_params.exclusive }}"
+      when: account.secret_type == "ssh_key"
 
     - name: "Set {{ account.username }} sudo setting"
       ansible.builtin.lineinfile:
@@ -32,61 +67,9 @@
         line: "{{ account.username + ' ALL=(ALL) NOPASSWD: ' + params.sudo }}"
         validate: visudo -cf %s
       when:
-        - user_info.msg is defined or params.modify_sudo
+        - user_info.failed
         - params.sudo
 
-    - name: "Change {{ account.username }} password"
-      ansible.builtin.user:
-        name: "{{ account.username }}"
-        password: "{{ account.secret | password_hash('des') }}"
-        update_password: always
-      ignore_errors: true
-      register: change_secret_result
-      when: account.secret_type == "password"
-
-    - name: "Get home directory for {{ account.username }}"
-      ansible.builtin.shell: "getent passwd {{ account.username }} | cut -d: -f6"
-      register: home_dir
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Check if home directory exists for {{ account.username }}"
-      ansible.builtin.stat:
-        path: "{{ home_dir.stdout.strip() }}"
-      register: home_dir_stat
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Ensure {{ account.username }} home directory exists"
-      ansible.builtin.file:
-        path: "{{ home_dir.stdout.strip() }}"
-        state: directory
-        owner: "{{ account.username }}"
-        group: "{{ account.username }}"
-        mode: '0750'
-      when:
-        - account.secret_type == "ssh_key"
-        - home_dir_stat.stat.exists == false
-      ignore_errors: yes
-
-    - name: Remove jumpserver ssh key
-      ansible.builtin.lineinfile:
-        dest: "{{ home_dir.stdout.strip() }}/.ssh/authorized_keys"
-        regexp: "{{ ssh_params.regexp }}"
-        state: absent
-      when:
-        - account.secret_type == "ssh_key"
-        - ssh_params.strategy == "set_jms"
-      ignore_errors: yes
-
-    - name: "Change {{ account.username }} SSH key"
-      ansible.builtin.authorized_key:
-        user: "{{ account.username }}"
-        key: "{{ account.secret }}"
-        exclusive: "{{ ssh_params.exclusive }}"
-      register: change_secret_result
-      when: account.secret_type == "ssh_key"
-
     - name: Refresh connection
       ansible.builtin.meta: reset_connection
 
@@ -96,16 +79,14 @@
         login_password: "{{ account.secret }}"
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         become: "{{ account.become.ansible_become | default(False) }}"
         become_method: su
         become_user: "{{ account.become.ansible_user | default('') }}"
         become_password: "{{ account.become.ansible_password | default('') }}"
         become_private_key_path: "{{ account.become.ansible_ssh_private_key_file | default(None) }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "password"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "password"
       delegate_to: localhost
 
     - name: "Verify {{ account.username }} SSH KEY (paramiko)"
@@ -114,9 +95,7 @@
         login_port: "{{ jms_asset.port }}"
         login_user: "{{ account.username }}"
         login_private_key_path: "{{ account.private_key_path  }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "ssh_key"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "ssh_key"
       delegate_to: localhost

apps/accounts/automations/change_secret/host/aix/manifest.yml

@@ -5,12 +5,6 @@ type:
   - AIX
 method: change_secret
 params:
-  - name: modify_sudo
-    type: bool
-    label: "{{ 'Modify sudo label' | trans }}"
-    default: False
-    help_text: "{{ 'Modify params sudo help text' | trans }}"
-
   - name: sudo
     type: str
     label: 'Sudo'
@@ -34,23 +28,12 @@ params:
     default: ''
     help_text: "{{ 'Params groups help text' | trans }}"
 
-  - name: uid
-    type: str
-    label: "{{ 'Params uid label' | trans }}"
-    default: ''
-    help_text: "{{ 'Params uid help text' | trans }}"
-
 i18n:
   AIX account change secret:
     zh: '使用 Ansible 模块 user 执行账号改密 (DES)'
     ja: 'Ansible user モジュールを使用してアカウントのパスワード変更 (DES)'
     en: 'Using Ansible module user to change account secret (DES)'
 
-  Modify params sudo help text:
-    zh: '如果用户存在，可以修改sudo权限'
-    ja: 'ユーザーが存在する場合、sudo権限を変更できます'
-    en: 'If the user exists, sudo permissions can be modified'
-
   Params sudo help text:
     zh: '使用逗号分隔多个命令，如: /bin/whoami,/sbin/ifconfig'
     ja: 'コンマで区切って複数のコマンドを入力してください。例: /bin/whoami,/sbin/ifconfig'
@@ -66,16 +49,6 @@ i18n:
     ja: 'グループを入力してください。複数のグループはコンマで区切ってください（既存のグループを入力してください）'
     en: 'Please enter the group. Multiple groups are separated by commas (please enter the existing group)'
 
-  Params uid help text:
-    zh: '请输入用户ID'
-    ja: 'ユーザーIDを入力してください'
-    en: 'Please enter the user ID'
-
-  Modify sudo label:
-    zh: '修改 sudo 权限'
-    ja: 'sudo 権限を変更'
-    en: 'Modify sudo'
-
   Params home label:
     zh: '家目录'
     ja: 'ホームディレクトリ'
@@ -86,7 +59,3 @@ i18n:
     ja: 'グループ'
     en: 'Groups'
 
-  Params uid label:
-    zh: '用户ID'
-    ja: 'ユーザーID'
-    en: 'User ID'

apps/accounts/automations/change_secret/host/posix/main.yml

@@ -9,20 +9,55 @@
         database: passwd
         key: "{{ account.username }}"
       register: user_info
-      failed_when: false
-      changed_when: false
+      ignore_errors: yes  # 忽略错误，如果用户不存在时不会导致playbook失败
 
     - name: "Add {{ account.username }} user"
       ansible.builtin.user:
         name: "{{ account.username }}"
-        uid: "{{ params.uid | int if params.uid | length > 0 else omit }}"
-        shell: "{{ params.shell if params.shell | length > 0 else omit }}"
-        home: "{{ params.home if params.home | length > 0 else '/home/' + account.username }}"
-        groups: "{{ params.groups if params.groups | length > 0 else omit }}"
-        append: "{{ true if params.groups | length > 0 else false }}"
+        shell: "{{ params.shell }}"
+        home: "{{ params.home | default('/home/' + account.username, true) }}"
+        groups: "{{ params.groups }}"
         expires: -1
         state: present
-      when: user_info.msg is defined
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} group"
+      ansible.builtin.group:
+        name: "{{ account.username }}"
+        state: present
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} user to group"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        groups: "{{ params.groups }}"
+      when:
+        - user_info.failed
+        - params.groups
+
+    - name: "Change {{ account.username }} password"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        password: "{{ account.secret | password_hash('sha512') }}"
+        update_password: always
+      ignore_errors: true
+      when: account.secret_type == "password"
+
+    - name: remove jumpserver ssh key
+      ansible.builtin.lineinfile:
+        dest: "{{ ssh_params.dest }}"
+        regexp: "{{ ssh_params.regexp }}"
+        state: absent
+      when:
+        - account.secret_type == "ssh_key"
+        - ssh_params.strategy == "set_jms"
+
+    - name: "Change {{ account.username }} SSH key"
+      ansible.builtin.authorized_key:
+        user: "{{ account.username }}"
+        key: "{{ account.secret }}"
+        exclusive: "{{ ssh_params.exclusive }}"
+      when: account.secret_type == "ssh_key"
 
     - name: "Set {{ account.username }} sudo setting"
       ansible.builtin.lineinfile:
@@ -32,61 +67,9 @@
         line: "{{ account.username + ' ALL=(ALL) NOPASSWD: ' + params.sudo }}"
         validate: visudo -cf %s
       when:
-        - user_info.msg is defined or params.modify_sudo
+        - user_info.failed
         - params.sudo
 
-    - name: "Change {{ account.username }} password"
-      ansible.builtin.user:
-        name: "{{ account.username }}"
-        password: "{{ account.secret | password_hash('sha512') }}"
-        update_password: always
-      ignore_errors: true
-      register: change_secret_result
-      when: account.secret_type == "password"
-
-    - name: "Get home directory for {{ account.username }}"
-      ansible.builtin.shell: "getent passwd {{ account.username }} | cut -d: -f6"
-      register: home_dir
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Check if home directory exists for {{ account.username }}"
-      ansible.builtin.stat:
-        path: "{{ home_dir.stdout.strip() }}"
-      register: home_dir_stat
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Ensure {{ account.username }} home directory exists"
-      ansible.builtin.file:
-        path: "{{ home_dir.stdout.strip() }}"
-        state: directory
-        owner: "{{ account.username }}"
-        group: "{{ account.username }}"
-        mode: '0750'
-      when:
-        - account.secret_type == "ssh_key"
-        - home_dir_stat.stat.exists == false
-      ignore_errors: yes
-
-    - name: Remove jumpserver ssh key
-      ansible.builtin.lineinfile:
-        dest: "{{ home_dir.stdout.strip() }}/.ssh/authorized_keys"
-        regexp: "{{ ssh_params.regexp }}"
-        state: absent
-      when:
-        - account.secret_type == "ssh_key"
-        - ssh_params.strategy == "set_jms"
-      ignore_errors: yes
-
-    - name: "Change {{ account.username }} SSH key"
-      ansible.builtin.authorized_key:
-        user: "{{ account.username }}"
-        key: "{{ account.secret }}"
-        exclusive: "{{ ssh_params.exclusive }}"
-      register: change_secret_result
-      when: account.secret_type == "ssh_key"
-
     - name: Refresh connection
       ansible.builtin.meta: reset_connection
 
@@ -96,16 +79,14 @@
         login_password: "{{ account.secret }}"
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         become: "{{ account.become.ansible_become | default(False) }}"
         become_method: su
         become_user: "{{ account.become.ansible_user | default('') }}"
         become_password: "{{ account.become.ansible_password | default('') }}"
         become_private_key_path: "{{ account.become.ansible_ssh_private_key_file | default(None) }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "password"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "password"
       delegate_to: localhost
 
     - name: "Verify {{ account.username }} SSH KEY (paramiko)"
@@ -114,9 +95,7 @@
         login_port: "{{ jms_asset.port }}"
         login_user: "{{ account.username }}"
         login_private_key_path: "{{ account.private_key_path  }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "ssh_key"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "ssh_key"
       delegate_to: localhost

apps/accounts/automations/change_secret/host/posix/manifest.yml

@@ -6,12 +6,6 @@ type:
   - linux
 method: change_secret
 params:
-  - name: modify_sudo
-    type: bool
-    label: "{{ 'Modify sudo label' | trans }}"
-    default: False
-    help_text: "{{ 'Modify params sudo help text' | trans }}"
-
   - name: sudo
     type: str
     label: 'Sudo'
@@ -36,23 +30,12 @@ params:
     default: ''
     help_text: "{{ 'Params groups help text' | trans }}"
 
-  - name: uid
-    type: str
-    label: "{{ 'Params uid label' | trans }}"
-    default: ''
-    help_text: "{{ 'Params uid help text' | trans }}"
-
 i18n:
   Posix account change secret:
     zh: '使用 Ansible 模块 user 执行账号改密 (SHA512)'
     ja: 'Ansible user モジュールを使用して アカウントのパスワード変更 (SHA512)'
     en: 'Using Ansible module user to change account secret (SHA512)'
 
-  Modify params sudo help text:
-    zh: '如果用户存在，可以修改sudo权限'
-    ja: 'ユーザーが存在する場合、sudo権限を変更できます'
-    en: 'If the user exists, sudo permissions can be modified'
-
   Params sudo help text:
     zh: '使用逗号分隔多个命令，如: /bin/whoami,/sbin/ifconfig'
     ja: 'コンマで区切って複数のコマンドを入力してください。例: /bin/whoami,/sbin/ifconfig'
@@ -68,16 +51,6 @@ i18n:
     ja: 'グループを入力してください。複数のグループはコンマで区切ってください（既存のグループを入力してください）'
     en: 'Please enter the group. Multiple groups are separated by commas (please enter the existing group)'
 
-  Params uid help text:
-    zh: '请输入用户ID'
-    ja: 'ユーザーIDを入力してください'
-    en: 'Please enter the user ID'
-
-  Modify sudo label:
-    zh: '修改 sudo 权限'
-    ja: 'sudo 権限を変更'
-    en: 'Modify sudo'
-
   Params home label:
     zh: '家目录'
     ja: 'ホームディレクトリ'
@@ -88,7 +61,3 @@ i18n:
     ja: 'グループ'
     en: 'Groups'
 
-  Params uid label:
-    zh: '用户ID'
-    ja: 'ユーザーID'
-    en: 'User ID'

apps/accounts/automations/change_secret/host/windows/main.yml

@@ -4,6 +4,10 @@
     - name: Test privileged account
       ansible.windows.win_ping:
 
+#    - name: Print variables
+#      debug:
+#        msg: "Username: {{ account.username }}, Password: {{ account.secret }}"
+
     - name: Change password
       ansible.windows.win_user:
         fullname: "{{ account.username}}"
@@ -24,4 +28,4 @@
       vars:
         ansible_user: "{{ account.username }}"
         ansible_password: "{{ account.secret }}"
-      when: account.secret_type == "password" and check_conn_after_change
+      when: account.secret_type == "password"

apps/accounts/automations/change_secret/host/windows_ad/main.yml

@@ -1,27 +0,0 @@
-- hosts: demo
-  gather_facts: no
-  tasks:
-    - name: Test privileged account
-      ansible.windows.win_ping:
-
-    - name: Change password
-      community.windows.win_domain_user:
-        name: "{{ account.username }}"
-        password: "{{ account.secret }}"
-        update_password: always
-        password_never_expires: yes
-        state: present
-        groups: "{{ params.groups }}"
-        groups_action: add
-      ignore_errors: true
-      when: account.secret_type == "password"
-
-    - name: Refresh connection
-      ansible.builtin.meta: reset_connection
-
-    - name: Verify password
-      ansible.windows.win_ping:
-      vars:
-        ansible_user: "{{ account.full_username }}"
-        ansible_password: "{{ account.secret }}"
-      when: account.secret_type == "password" and check_conn_after_change

apps/accounts/automations/change_secret/host/windows_ad/manifest.yml

@@ -1,27 +0,0 @@
-id: change_secret_ad_windows
-name: "{{ 'Windows account change secret' | trans }}"
-version: 1
-method: change_secret
-category:
-  - ds
-type:
-  - windows_ad
-params:
-  - name: groups
-    type: str
-    label: '用户组'
-    default: 'Users,Remote Desktop Users'
-    help_text: "{{ 'Params groups help text' | trans }}"
-
-
-i18n:
-  Windows account change secret:
-    zh: '使用 Ansible 模块 win_domain_user 执行 Windows 账号改密'
-    ja: 'Ansible win_domain_user モジュールを使用して Windows アカウントのパスワード変更'
-    en: 'Using Ansible module win_domain_user to change Windows account secret'
-
-  Params groups help text:
-    zh: '请输入用户组，多个用户组使用逗号分隔（需填写已存在的用户组）'
-    ja: 'グループを入力してください。複数のグループはコンマで区切ってください（既存のグループを入力してください）'
-    en: 'Please enter the group. Multiple groups are separated by commas (please enter the existing group)'
-

apps/accounts/automations/change_secret/host/windows_rdp_verify/main.yml

@@ -4,6 +4,10 @@
     - name: Test privileged account
       ansible.windows.win_ping:
 
+#    - name: Print variables
+#      debug:
+#        msg: "Username: {{ account.username }}, Password: {{ account.secret }}"
+
     - name: Change password
       ansible.windows.win_user:
         fullname: "{{ account.username}}"
@@ -21,11 +25,11 @@
 
     - name: Verify password (pyfreerdp)
       rdp_ping:
-        login_host: "{{ jms_asset.origin_address }}"
+        login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.protocols | selectattr('name', 'equalto', 'rdp') | map(attribute='port') | first }}"
         login_user: "{{ account.username }}"
         login_password: "{{ account.secret }}"
         login_secret_type: "{{ account.secret_type }}"
-        gateway_args: "{{ jms_gateway | default({}) }}"
-      when: account.secret_type == "password" and check_conn_after_change
+        login_private_key_path: "{{ account.private_key_path }}"
+      when: account.secret_type == "password"
       delegate_to: localhost

apps/accounts/automations/change_secret/manager.py

@@ -1,57 +1,208 @@
 import os
 import time
+from copy import deepcopy
 
 from django.conf import settings
+from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 from xlsxwriter import Workbook
 
-from accounts.const import (
-    AutomationTypes, SecretStrategy, ChangeSecretRecordStatusChoice
-)
+from accounts.const import AutomationTypes, SecretType, SSHKeyStrategy, SecretStrategy, ChangeSecretRecordStatusChoice
 from accounts.models import ChangeSecretRecord
-from accounts.notifications import ChangeSecretExecutionTaskMsg, ChangeSecretReportMsg
+from accounts.notifications import ChangeSecretExecutionTaskMsg, ChangeSecretFailedMsg
 from accounts.serializers import ChangeSecretRecordBackUpSerializer
+from assets.const import HostTypes
 from common.utils import get_logger
 from common.utils.file import encrypt_and_compress_zip_file
 from common.utils.timezone import local_now_filename
-from ..base.manager import BaseChangeSecretPushManager
+from users.models import User
+from ..base.manager import AccountBasePlaybookManager
+from ...utils import SecretGenerator
 
 logger = get_logger(__name__)
 
 
-class ChangeSecretManager(BaseChangeSecretPushManager):
+class ChangeSecretManager(AccountBasePlaybookManager):
     ansible_account_prefer = ''
 
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.record_map = self.execution.snapshot.get('record_map', {})
+        self.secret_type = self.execution.snapshot.get('secret_type')
+        self.secret_strategy = self.execution.snapshot.get(
+            'secret_strategy', SecretStrategy.custom
+        )
+        self.ssh_key_change_strategy = self.execution.snapshot.get(
+            'ssh_key_change_strategy', SSHKeyStrategy.add
+        )
+        self.account_ids = self.execution.snapshot['accounts']
+        self.name_recorder_mapper = {}  # 做个映射，方便后面处理
+
     @classmethod
     def method_type(cls):
         return AutomationTypes.change_secret
 
-    def gen_account_inventory(self, account, asset, h, path_dir):
-        record = self.get_or_create_record(asset, account, h['name'])
-        new_secret, private_key_path = self.handle_ssh_secret(account.secret_type, record.new_secret, path_dir)
-        h = self.gen_inventory(h, account, new_secret, private_key_path, asset)
-        return h
+    def get_ssh_params(self, account, secret, secret_type):
+        kwargs = {}
+        if secret_type != SecretType.SSH_KEY:
+            return kwargs
+        kwargs['strategy'] = self.ssh_key_change_strategy
+        kwargs['exclusive'] = 'yes' if kwargs['strategy'] == SSHKeyStrategy.set else 'no'
 
-    def get_or_create_record(self, asset, account, name):
-        asset_account_id = f'{asset.id}-{account.id}'
+        if kwargs['strategy'] == SSHKeyStrategy.set_jms:
+            username = account.username
+            path = f'/{username}' if username == "root" else f'/home/{username}'
+            kwargs['dest'] = f'{path}/.ssh/authorized_keys'
+            kwargs['regexp'] = '.*{}$'.format(secret.split()[2].strip())
+        return kwargs
 
-        if asset_account_id in self.record_map:
-            record_id = self.record_map[asset_account_id]
-            recorder = ChangeSecretRecord.objects.filter(id=record_id).first()
-        else:
-            new_secret = self.get_secret(account)
-            recorder = self.create_record(asset, account, new_secret)
-
-        self.name_recorder_mapper[name] = recorder
-        return recorder
-
-    def create_record(self, asset, account, new_secret):
-        recorder = ChangeSecretRecord(
-            asset=asset, account=account, execution=self.execution,
-            old_secret=account.secret, new_secret=new_secret,
-            comment=f'{account.username}@{asset.address}'
+    def secret_generator(self, secret_type):
+        return SecretGenerator(
+            self.secret_strategy, secret_type,
+            self.execution.snapshot.get('password_rules')
         )
-        return recorder
+
+    def get_secret(self, secret_type):
+        if self.secret_strategy == SecretStrategy.custom:
+            return self.execution.snapshot['secret']
+        else:
+            return self.secret_generator(secret_type).get_secret()
+
+    def get_accounts(self, privilege_account):
+        if not privilege_account:
+            print(f'not privilege account')
+            return []
+
+        asset = privilege_account.asset
+        accounts = asset.accounts.all()
+        accounts = accounts.filter(id__in=self.account_ids)
+        if self.secret_type:
+            accounts = accounts.filter(secret_type=self.secret_type)
+
+        if settings.CHANGE_AUTH_PLAN_SECURE_MODE_ENABLED:
+            accounts = accounts.filter(privileged=False).exclude(
+                username__in=['root', 'administrator', privilege_account.username]
+            )
+        return accounts
+
+    def host_callback(
+            self, host, asset=None, account=None,
+            automation=None, path_dir=None, **kwargs
+    ):
+        host = super().host_callback(
+            host, asset=asset, account=account, automation=automation,
+            path_dir=path_dir, **kwargs
+        )
+        if host.get('error'):
+            return host
+
+        accounts = self.get_accounts(account)
+        error_msg = _("No pending accounts found")
+        if not accounts:
+            print(f'{asset}: {error_msg}')
+            return []
+
+        records = []
+        inventory_hosts = []
+        if asset.type == HostTypes.WINDOWS and self.secret_type == SecretType.SSH_KEY:
+            print(f'Windows {asset} does not support ssh key push')
+            return inventory_hosts
+
+        host['ssh_params'] = {}
+        for account in accounts:
+            h = deepcopy(host)
+            secret_type = account.secret_type
+            h['name'] += '(' + account.username + ')'
+            if self.secret_type is None:
+                new_secret = account.secret
+            else:
+                new_secret = self.get_secret(secret_type)
+
+            if new_secret is None:
+                print(f'new_secret is None, account: {account}')
+                continue
+
+            asset_account_id = f'{asset.id}-{account.id}'
+            if asset_account_id not in self.record_map:
+                recorder = ChangeSecretRecord(
+                    asset=asset, account=account, execution=self.execution,
+                    old_secret=account.secret, new_secret=new_secret,
+                )
+                records.append(recorder)
+            else:
+                record_id = self.record_map[asset_account_id]
+                try:
+                    recorder = ChangeSecretRecord.objects.get(id=record_id)
+                except ChangeSecretRecord.DoesNotExist:
+                    print(f"Record {record_id} not found")
+                    continue
+
+            self.name_recorder_mapper[h['name']] = recorder
+
+            private_key_path = None
+            if secret_type == SecretType.SSH_KEY:
+                private_key_path = self.generate_private_key_path(new_secret, path_dir)
+                new_secret = self.generate_public_key(new_secret)
+
+            h['ssh_params'].update(self.get_ssh_params(account, new_secret, secret_type))
+            h['account'] = {
+                'name': account.name,
+                'username': account.username,
+                'secret_type': secret_type,
+                'secret': account.escape_jinja2_syntax(new_secret),
+                'private_key_path': private_key_path,
+                'become': account.get_ansible_become_auth(),
+            }
+            if asset.platform.type == 'oracle':
+                h['account']['mode'] = 'sysdba' if account.privileged else None
+            inventory_hosts.append(h)
+        ChangeSecretRecord.objects.bulk_create(records)
+        return inventory_hosts
+
+    def on_host_success(self, host, result):
+        recorder = self.name_recorder_mapper.get(host)
+        if not recorder:
+            return
+        recorder.status = ChangeSecretRecordStatusChoice.success.value
+        recorder.date_finished = timezone.now()
+
+        account = recorder.account
+        if not account:
+            print("Account not found, deleted ?")
+            return
+        account.secret = recorder.new_secret
+        account.date_updated = timezone.now()
+
+        max_retries = 3
+        retry_count = 0
+
+        while retry_count < max_retries:
+            try:
+                recorder.save()
+                account.save(update_fields=['secret', 'version', 'date_updated'])
+                break
+            except Exception as e:
+                retry_count += 1
+                if retry_count == max_retries:
+                    self.on_host_error(host, str(e), result)
+                else:
+                    print(f'retry {retry_count} times for {host} recorder save error: {e}')
+                    time.sleep(1)
+
+    def on_host_error(self, host, error, result):
+        recorder = self.name_recorder_mapper.get(host)
+        if not recorder:
+            return
+        recorder.status = ChangeSecretRecordStatusChoice.failed.value
+        recorder.date_finished = timezone.now()
+        recorder.error = error
+        try:
+            recorder.save()
+        except Exception as e:
+            print(f"\033[31m Save {host} recorder error: {e} \033[0m\n")
+
+    def on_runner_failed(self, runner, e):
+        logger.error("Account error: ", e)
 
     def check_secret(self):
         if self.secret_strategy == SecretStrategy.custom \
@@ -69,39 +220,44 @@ class ChangeSecretManager(BaseChangeSecretPushManager):
             else:
                 failed += 1
             total += 1
+
         summary = _('Success: %s, Failed: %s, Total: %s') % (succeed, failed, total)
         return summary
 
-    def print_summary(self):
-        recorders = list(self.name_recorder_mapper.values())
-        summary = self.get_summary(recorders)
-        print('\n\n' + '-' * 80)
-        plan_execution_end = _('Plan execution end')
-        print('{} {}\n'.format(plan_execution_end, local_now_filename()))
-        time_cost = _('Duration')
-        print('{}: {}s'.format(time_cost, self.duration))
-        print(summary)
-
-    def send_report_if_need(self, *args, **kwargs):
+    def run(self, *args, **kwargs):
         if self.secret_type and not self.check_secret():
             return
-
+        super().run(*args, **kwargs)
         recorders = list(self.name_recorder_mapper.values())
+        summary = self.get_summary(recorders)
+        print(summary, end='')
+
         if self.record_map:
             return
 
+        failed_recorders = [
+            r for r in recorders
+            if r.status == ChangeSecretRecordStatusChoice.failed.value
+        ]
+
         recipients = self.execution.recipients
+        recipients = User.objects.filter(id__in=list(recipients.keys()))
         if not recipients:
             return
 
-        context = self.get_report_context()
-        for user in recipients:
-            ChangeSecretReportMsg(user, context).publish()
+        if failed_recorders:
+            name = self.execution.snapshot.get('name')
+            execution_id = str(self.execution.id)
+            _ids = [r.id for r in failed_recorders]
+            asset_account_errors = ChangeSecretRecord.objects.filter(
+                id__in=_ids).values_list('asset__name', 'account__username', 'error')
+
+            for user in recipients:
+                ChangeSecretFailedMsg(name, execution_id, user, asset_account_errors).publish()
 
         if not recorders:
             return
 
-        summary = self.get_summary(recorders)
         self.send_recorder_mail(recipients, recorders, summary)
 
     def send_recorder_mail(self, recipients, recorders, summary):
@@ -138,6 +294,3 @@ class ChangeSecretManager(BaseChangeSecretPushManager):
                 ws.write_string(row_index, col_index, col_data)
         wb.close()
         return True
-
-    def get_report_template(self):
-        return "accounts/change_secret_report.html"

apps/accounts/automations/check_account/add_to_leak_password.py

@@ -1,78 +0,0 @@
-#!/usr/bin/env python
-#
-import re
-import sqlite3
-import sys
-
-
-def is_weak_password(password):
-    if len(password) < 8:
-        return True
-
-    # 判断是否只有一种字符类型
-    if password.isdigit() or password.isalpha():
-        return True
-
-    # 判断是否只包含数字或字母
-    if password.islower() or password.isupper():
-        return True
-
-    # 判断是否包含常见弱密码
-    common_passwords = ["123456", "password", "12345678", "qwerty", "abc123"]
-    if password.lower() in common_passwords:
-        return True
-
-    # 正则表达式判断字符多样性（数字、字母、特殊字符）
-    if (
-            not re.search(r"[A-Za-z]", password)
-            or not re.search(r"[0-9]", password)
-            or not re.search(r"[\W_]", password)
-    ):
-        return True
-    return False
-
-
-def parse_it(fname):
-    count = 0
-    lines = []
-    with open(fname, 'rb') as f:
-        for line in f:
-            try:
-                line = line.decode().strip()
-            except UnicodeDecodeError:
-                continue
-
-            if len(line) > 32:
-                continue
-
-            if is_weak_password(line):
-                continue
-
-            lines.append(line)
-            count += 0
-            print(line)
-    return lines
-
-
-def insert_to_db(lines):
-    conn = sqlite3.connect('./leak_passwords.db')
-    cursor = conn.cursor()
-    create_table_sql = '''
-        CREATE TABLE IF NOT EXISTS passwords (
-            id INTEGER PRIMARY KEY AUTOINCREMENT,
-            password CHAR(32)
-        )
-    '''
-    create_index_sql = 'CREATE INDEX IF NOT EXISTS idx_password ON passwords(password)'
-    cursor.execute(create_table_sql)
-    cursor.execute(create_index_sql)
-
-    for line in lines:
-        cursor.execute('INSERT INTO passwords (password) VALUES (?)', [line])
-    conn.commit()
-
-
-if __name__ == '__main__':
-    filename = sys.argv[1]
-    lines = parse_it(filename)
-    insert_to_db(lines)

apps/accounts/automations/check_account/manager.py

@@ -1,283 +0,0 @@
-import hashlib
-import os
-import re
-import sqlite3
-import uuid
-
-from django.conf import settings
-from django.utils import timezone
-from django.utils.translation import gettext_lazy as _
-
-from accounts.models import Account, AccountRisk, RiskChoice
-from assets.automations.base.manager import BaseManager
-from common.const import ConfirmOrIgnore
-from common.decorators import bulk_create_decorator, bulk_update_decorator
-
-
-@bulk_create_decorator(AccountRisk)
-def create_risk(data):
-    return AccountRisk(**data)
-
-
-@bulk_update_decorator(AccountRisk, update_fields=["details", "status"])
-def update_risk(risk):
-    return risk
-
-
-class BaseCheckHandler:
-    risk = ''
-
-    def __init__(self, assets):
-        self.assets = assets
-
-    def check(self, account):
-        pass
-
-    def clean(self):
-        pass
-
-
-class CheckSecretHandler(BaseCheckHandler):
-    risk = RiskChoice.weak_password
-
-    @staticmethod
-    def is_weak_password(password):
-        # 判断密码长度
-        if len(password) < 8:
-            return True
-
-        # 判断是否只有一种字符类型
-        if password.isdigit() or password.isalpha():
-            return True
-
-        # 判断是否只包含数字或字母
-        if password.islower() or password.isupper():
-            return True
-
-        # 判断是否包含常见弱密码
-        common_passwords = ["123456", "password", "12345678", "qwerty", "abc123"]
-        if password.lower() in common_passwords:
-            return True
-
-        # 正则表达式判断字符多样性（数字、字母、特殊字符）
-        if (
-                not re.search(r"[A-Za-z]", password)
-                or not re.search(r"[0-9]", password)
-                or not re.search(r"[\W_]", password)
-        ):
-            return True
-        return False
-
-    def check(self, account):
-        if not account.secret:
-            return False
-        return self.is_weak_password(account.secret)
-
-
-class CheckRepeatHandler(BaseCheckHandler):
-    risk = RiskChoice.repeated_password
-
-    def __init__(self, assets):
-        super().__init__(assets)
-        self.path, self.conn, self.cursor = self.init_repeat_check_db()
-        self.add_password_for_check_repeat()
-
-    @staticmethod
-    def init_repeat_check_db():
-        path = os.path.join('/tmp', 'accounts_' + str(uuid.uuid4()) + '.db')
-        sql = """
-        CREATE TABLE IF NOT EXISTS accounts (
-            id INTEGER PRIMARY KEY AUTOINCREMENT,
-            digest CHAR(32)
-        )
-        """
-        index = "CREATE INDEX IF NOT EXISTS idx_digest ON accounts(digest)"
-        conn = sqlite3.connect(path)
-        cursor = conn.cursor()
-        cursor.execute(sql)
-        cursor.execute(index)
-        return path, conn, cursor
-
-    def check(self, account):
-        if not account.secret:
-            return False
-
-        digest = self.digest(account.secret)
-        sql = 'SELECT COUNT(*) FROM accounts WHERE digest = ?'
-        self.cursor.execute(sql, [digest])
-        result = self.cursor.fetchone()
-        if not result:
-            return False
-        return result[0] > 1
-
-    @staticmethod
-    def digest(secret):
-        return hashlib.md5(secret.encode()).hexdigest()
-
-    def add_password_for_check_repeat(self):
-        accounts = Account.objects.all().only('id', '_secret', 'secret_type')
-        sql = "INSERT INTO accounts (digest) VALUES (?)"
-
-        for account in accounts:
-            secret = account.secret
-            if not secret:
-                continue
-            digest = self.digest(secret)
-            self.cursor.execute(sql, [digest])
-        self.conn.commit()
-
-    def clean(self):
-        self.cursor.close()
-        self.conn.close()
-        os.remove(self.path)
-
-
-class CheckLeakHandler(BaseCheckHandler):
-    risk = RiskChoice.leaked_password
-
-    def __init__(self, *args):
-        super().__init__(*args)
-        self.conn, self.cursor = self.init_leak_password_db()
-
-    @staticmethod
-    def init_leak_password_db():
-        db_path = os.path.join(
-            settings.APPS_DIR, 'accounts', 'automations',
-            'check_account', 'leak_passwords.db'
-        )
-
-        if settings.LEAK_PASSWORD_DB_PATH and os.path.isfile(settings.LEAK_PASSWORD_DB_PATH):
-            db_path = settings.LEAK_PASSWORD_DB_PATH
-
-        db_conn = sqlite3.connect(db_path)
-        db_cursor = db_conn.cursor()
-        return db_conn, db_cursor
-
-    def check(self, account):
-        if not account.secret:
-            return False
-
-        sql = 'SELECT 1 FROM passwords WHERE password = ? LIMIT 1'
-        self.cursor.execute(sql, (account.secret,))
-        leak = self.cursor.fetchone() is not None
-        return leak
-
-    def clean(self):
-        self.cursor.close()
-        self.conn.close()
-
-
-class CheckAccountManager(BaseManager):
-    batch_size = 100
-    tmpl = 'Checked the status of account %s: %s'
-
-    def __init__(self, execution):
-        super().__init__(execution)
-        self.assets = []
-        self.batch_risks = []
-        self.handlers = []
-
-    def add_risk(self, risk, account):
-        self.summary[risk] += 1
-        self.result[risk].append({
-            'asset': str(account.asset), 'username': account.username,
-        })
-        risk_obj = {'account': account, 'risk': risk}
-        self.batch_risks.append(risk_obj)
-
-    def commit_risks(self, assets):
-        account_risks = AccountRisk.objects.filter(asset__in=assets)
-        ori_risk_map = {}
-
-        for risk in account_risks:
-            key = f'{risk.account_id}_{risk.risk}'
-            ori_risk_map[key] = risk
-
-        now = timezone.now().isoformat()
-        for d in self.batch_risks:
-            account = d["account"]
-            key = f'{account.id}_{d["risk"]}'
-            origin_risk = ori_risk_map.get(key)
-
-            if origin_risk and origin_risk.status != ConfirmOrIgnore.pending:
-                details = origin_risk.details or []
-                details.append({"datetime": now, 'type': 'refind'})
-
-                if len(details) > 10:
-                    details = [*details[:5], *details[-5:]]
-
-                origin_risk.details = details
-                origin_risk.status = ConfirmOrIgnore.pending
-                update_risk(origin_risk)
-            else:
-                create_risk({
-                    "account": account,
-                    "asset": account.asset,
-                    "username": account.username,
-                    "risk": d["risk"],
-                    "details": [{"datetime": now, 'type': 'init'}],
-                })
-
-    def pre_run(self):
-        super().pre_run()
-        self.assets = self.execution.get_all_assets()
-
-    def batch_check(self, handler):
-        print("Engine: {}".format(handler.__class__.__name__))
-        for i in range(0, len(self.assets), self.batch_size):
-            _assets = self.assets[i: i + self.batch_size]
-            accounts = Account.objects.filter(asset__in=_assets)
-
-            print("Start to check accounts: {}".format(len(accounts)))
-
-            for account in accounts:
-                error = handler.check(account)
-                msg = handler.risk if error else 'ok'
-
-                print("Check: {} => {}".format(account, msg))
-                if not error:
-                    continue
-                self.add_risk(handler.risk, account)
-            self.commit_risks(_assets)
-
-    def do_run(self, *args, **kwargs):
-        engines = self.execution.snapshot.get("engines", [])
-        if engines == '__all__':
-            engines = ['check_account_secret', 'check_account_repeat', 'check_account_leak']
-
-        for engine in engines:
-            if engine == "check_account_secret":
-                handler = CheckSecretHandler(self.assets)
-            elif engine == "check_account_repeat":
-                handler = CheckRepeatHandler(self.assets)
-            elif engine == "check_account_leak":
-                handler = CheckLeakHandler(self.assets)
-            else:
-                print("Unknown engine: {}".format(engine))
-                continue
-
-            self.handlers.append(handler)
-            self.batch_check(handler)
-
-    def post_run(self):
-        super().post_run()
-        for handler in self.handlers:
-            handler.clean()
-
-    def get_report_subject(self):
-        return "Check account report of %s" % self.execution.id
-
-    def get_report_template(self):
-        return "accounts/check_account_report.html"
-
-    def print_summary(self):
-        tmpl = _("---\nSummary: \nok: {}, weak password: {}, leaked password: {}, "
-                 "repeated password: {}, no secret: {}, using time: {}s").format(
-            self.summary["ok"],
-            self.summary[RiskChoice.weak_password],
-            self.summary[RiskChoice.leaked_password],
-            self.summary[RiskChoice.repeated_password],
-            self.summary["no_secret"],
-            self.duration
-        )
-        print(tmpl)

apps/accounts/automations/endpoint.py

@@ -1,7 +1,6 @@
 from .backup_account.manager import AccountBackupManager
 from .change_secret.manager import ChangeSecretManager
-from .check_account.manager import CheckAccountManager
-from .gather_account.manager import GatherAccountsManager
+from .gather_accounts.manager import GatherAccountsManager
 from .push_account.manager import PushAccountManager
 from .remove_account.manager import RemoveAccountManager
 from .verify_account.manager import VerifyAccountManager
@@ -17,8 +16,8 @@ class ExecutionManager:
         AutomationTypes.remove_account: RemoveAccountManager,
         AutomationTypes.gather_accounts: GatherAccountsManager,
         AutomationTypes.verify_gateway_account: VerifyGatewayAccountManager,
-        AutomationTypes.check_account: CheckAccountManager,
-        AutomationTypes.backup_account: AccountBackupManager,
+        # TODO 后期迁移到自动化策略中
+        'backup_account': AccountBackupManager,
     }
 
     def __init__(self, execution):
@@ -27,6 +26,3 @@ class ExecutionManager:
 
     def run(self, *args, **kwargs):
         return self._runner.run(*args, **kwargs)
-
-    def __getattr__(self, item):
-        return getattr(self._runner, item)

apps/accounts/automations/gather_account/database/postgresql/main.yml

@@ -1,30 +0,0 @@
-- hosts: postgresql
-  gather_facts: no
-  vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
-    check_ssl: "{{ jms_asset.spec_info.use_ssl }}"
-    ca_cert: "{{ jms_asset.secret_info.ca_cert | default('') }}"
-    ssl_cert: "{{ jms_asset.secret_info.client_cert | default('') }}"
-    ssl_key: "{{ jms_asset.secret_info.client_key | default('') }}"
-
-  tasks:
-    - name: Get info
-      community.postgresql.postgresql_info:
-        login_user: "{{ jms_account.username }}"
-        login_password: "{{ jms_account.secret }}"
-        login_host: "{{ jms_asset.address }}"
-        login_port: "{{ jms_asset.port }}"
-        login_db: "{{ jms_asset.spec_info.db_name }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
-        filter: "roles"
-      register: db_info
-
-    - name: Define info by set_fact
-      set_fact:
-        info: "{{ db_info.roles }}"
-
-    - debug:
-        var: info

apps/accounts/automations/gather_account/database/sqlserver/main.yml

@@ -1,43 +0,0 @@
-- hosts: sqlserver
-  gather_facts: no
-  vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
-
-  tasks:
-    - name: Test SQLServer connection
-      community.general.mssql_script:
-        login_user: "{{ jms_account.username }}"
-        login_password: "{{ jms_account.secret }}"
-        login_host: "{{ jms_asset.address }}"
-        login_port: "{{ jms_asset.port }}"
-        name: '{{ jms_asset.spec_info.db_name }}'
-        script: |
-          SELECT 
-              l.name,
-              l.modify_date,
-              l.is_disabled,
-              l.create_date,
-              l.default_database_name,
-              LOGINPROPERTY(name, 'DaysUntilExpiration') AS days_until_expiration,
-              MAX(s.login_time) AS last_login_time
-          FROM 
-              sys.sql_logins l
-          LEFT JOIN 
-              sys.dm_exec_sessions s
-          ON 
-              l.name = s.login_name
-          WHERE 
-              s.is_user_process = 1 OR s.login_name IS NULL
-          GROUP BY 
-              l.name, l.create_date, l.modify_date, l.is_disabled, l.default_database_name
-          ORDER BY 
-              last_login_time DESC;
-        output: dict
-      register: db_info
-
-    - name: Define info by set_fact
-      set_fact:
-        info: "{{ db_info.query_results_dict }}"
-
-    - debug:
-        var: info

apps/accounts/automations/gather_account/database/sqlserver/manifest.yml

@@ -1,10 +0,0 @@
-id: gather_accounts_sqlserver
-name: "{{ 'SQLServer account gather' | trans }}"
-category: database
-type:
-  - sqlserver
-method: gather_accounts
-i18n:
-  SQLServer account gather:
-    zh: SQLServer 账号收集
-    ja: SQLServer アカウントの収集

apps/accounts/automations/gather_account/filter.py

@@ -1,270 +0,0 @@
-from datetime import datetime
-
-from django.utils import timezone
-
-__all__ = ['GatherAccountsFilter']
-
-
-def parse_date(date_str, default=None):
-    if not date_str:
-        return default
-    if date_str in ['Never', 'null']:
-        return default
-    formats = [
-        '%Y/%m/%d %H:%M:%S',
-        '%Y-%m-%dT%H:%M:%S',
-        '%Y-%m-%d %H:%M:%S',
-        '%d-%m-%Y %H:%M:%S',
-        '%Y/%m/%d',
-        '%d-%m-%Y',
-    ]
-    for fmt in formats:
-        try:
-            dt = datetime.strptime(date_str, fmt)
-            return timezone.make_aware(dt, timezone.get_current_timezone())
-        except ValueError:
-            continue
-    return default
-
-
-class GatherAccountsFilter:
-    def __init__(self, tp):
-        self.tp = tp
-
-    @staticmethod
-    def mysql_filter(info):
-        result = {}
-        for host, user_dict in info.items():
-            for username, user_info in user_dict.items():
-                password_last_changed = parse_date(user_info.get('password_last_changed'))
-                password_lifetime = user_info.get('password_lifetime')
-                user = {
-                    'username': username,
-                    'date_password_change': password_last_changed,
-                    'date_password_expired': password_last_changed + timezone.timedelta(
-                        days=password_lifetime) if password_last_changed and password_lifetime else None,
-                    'date_last_login': None,
-                    'groups': '',
-                }
-                result[username] = user
-        return result
-
-    @staticmethod
-    def postgresql_filter(info):
-        result = {}
-        for username, user_info in info.items():
-            user = {
-                'username': username,
-                'date_password_change': None,
-                'date_password_expired': parse_date(user_info.get('valid_until')),
-                'date_last_login': None,
-                'groups': '',
-            }
-            detail = {
-                'can_login': user_info.get('canlogin'),
-                'superuser': user_info.get('superuser'),
-            }
-            user['detail'] = detail
-            result[username] = user
-        return result
-
-    @staticmethod
-    def sqlserver_filter(info):
-        if not info:
-            return {}
-        result = {}
-        for user_info in info[0][0]:
-            days_until_expiration = user_info.get('days_until_expiration')
-            date_password_expired = timezone.now() + timezone.timedelta(
-                days=int(days_until_expiration)) if days_until_expiration else None
-            user = {
-                'username': user_info.get('name', ''),
-                'date_password_change': parse_date(user_info.get('modify_date')),
-                'date_password_expired': date_password_expired,
-                'date_last_login': parse_date(user_info.get('last_login_time')),
-                'groups': '',
-            }
-            detail = {
-                'create_date': user_info.get('create_date', ''),
-                'is_disabled': user_info.get('is_disabled', ''),
-                'default_database_name': user_info.get('default_database_name', ''),
-            }
-            user['detail'] = detail
-            result[user['username']] = user
-        return result
-
-    @staticmethod
-    def oracle_filter(info):
-        result = {}
-        for default_tablespace, users in info.items():
-            for username, user_info in users.items():
-                user = {
-                    'username': username,
-                    'date_password_change': parse_date(user_info.get('password_change_date')),
-                    'date_password_expired': parse_date(user_info.get('expiry_date')),
-                    'date_last_login': parse_date(user_info.get('last_login')),
-                    'groups': '',
-                }
-                detail = {
-                    'uid': user_info.get('user_id', ''),
-                    'create_date': user_info.get('created', ''),
-                    'account_status': user_info.get('account_status', ''),
-                    'default_tablespace': default_tablespace,
-                    'roles': user_info.get('roles', []),
-                    'privileges': user_info.get('privileges', []),
-                }
-                user['detail'] = detail
-                result[user['username']] = user
-        return result
-
-    @staticmethod
-    def posix_filter(info):
-        user_groups = info.pop('user_groups', [])
-        username_groups = {}
-        for line in user_groups:
-            if ':' not in line:
-                continue
-            username, groups = line.split(':', 1)
-            username_groups[username.strip()] = groups.strip()
-
-        user_sudo = info.pop('user_sudo', [])
-        username_sudo = {}
-        for line in user_sudo:
-            if ':' not in line:
-                continue
-            username, sudo = line.split(':', 1)
-            if not sudo.strip():
-                continue
-            username_sudo[username.strip()] = sudo.strip()
-
-        last_login = info.pop('last_login', '')
-        user_last_login = {}
-        for line in last_login:
-            if not line.strip() or ' ' not in line:
-                continue
-            username, login = line.split(' ', 1)
-            user_last_login[username] = login.split()
-
-        user_authorized = info.pop('user_authorized', [])
-        username_authorized = {}
-        for line in user_authorized:
-            if ':' not in line:
-                continue
-            username, authorized = line.split(':', 1)
-            username_authorized[username.strip()] = authorized.strip()
-
-        passwd_date = info.pop('passwd_date', [])
-        username_password_date = {}
-        for line in passwd_date:
-            if ':' not in line:
-                continue
-            username, password_date = line.split(':', 1)
-            username_password_date[username.strip()] = password_date.strip().split()
-
-        result = {}
-        users = info.pop('users', '')
-
-        for username in users:
-            if not username:
-                continue
-            user = dict()
-
-            login = user_last_login.get(username) or ''
-            if login and len(login) == 3:
-                user['address_last_login'] = login[0][:32]
-                try:
-                    login_date = timezone.datetime.fromisoformat(login[1])
-                    user['date_last_login'] = login_date
-                except ValueError:
-                    pass
-
-            start_date = timezone.make_aware(timezone.datetime(1970, 1, 1))
-            _password_date = username_password_date.get(username) or ''
-            if _password_date and len(_password_date) == 2:
-                if _password_date[0]:
-                    user['date_password_change'] = start_date + timezone.timedelta(days=int(_password_date[0]))
-                if _password_date[1]:
-                    user['date_password_expired'] = start_date + timezone.timedelta(days=int(_password_date[1]))
-            detail = {
-                'groups': username_groups.get(username) or '',
-                'sudoers': username_sudo.get(username) or '',
-                'authorized_keys': username_authorized.get(username) or ''
-            }
-            user['detail'] = detail
-            result[username] = user
-        return result
-
-    @staticmethod
-    def windows_filter(info):
-        result = {}
-        for user_details in info['user_details']:
-            user_info = {}
-            lines = user_details['stdout_lines']
-            for line in lines:
-                if not line.strip():
-                    continue
-                parts = line.split('  ', 1)
-                if len(parts) == 2:
-                    key, value = parts
-                    user_info[key.strip()] = value.strip()
-            detail = {'groups': user_info.get('Global Group memberships', ''), }
-
-            username = user_info.get('User name')
-            if not username:
-                continue
-
-            result[username] = {
-                'username': username,
-                'date_password_change': parse_date(user_info.get('Password last set')),
-                'date_password_expired': parse_date(user_info.get('Password expires')),
-                'date_last_login': parse_date(user_info.get('Last logon')),
-                'groups': detail,
-            }
-        return result
-
-    @staticmethod
-    def windows_ad_filter(info):
-        result = {}
-        for user_info in info['user_details']:
-            detail = {'groups': user_info.get('GlobalGroupMemberships', ''), }
-            username = user_info.get('SamAccountName')
-            if not username:
-                continue
-            result[username] = {
-                'username': username,
-                'date_password_change': parse_date(user_info.get('PasswordLastSet')),
-                'date_password_expired': parse_date(user_info.get('PasswordExpires')),
-                'date_last_login': parse_date(user_info.get('LastLogonDate')),
-                'groups': detail,
-            }
-        return result
-
-    @staticmethod
-    def mongodb_filter(info):
-        result = {}
-        for db, users in info.items():
-            for username, user_info in users.items():
-                user = {
-                    'username': username,
-                    'date_password_change': None,
-                    'date_password_expired': None,
-                    'date_last_login': None,
-                    'groups': '',
-                }
-                result['detail'] = {'db': db, 'roles': user_info.get('roles', [])}
-                result[username] = user
-        return result
-
-    def run(self, method_id_meta_mapper, info):
-        run_method_name = None
-        for k, v in method_id_meta_mapper.items():
-            if self.tp not in v['type']:
-                continue
-            run_method_name = k.replace(f'{v["method"]}_', '')
-
-        if not run_method_name:
-            return info
-
-        if hasattr(self, f'{run_method_name}_filter'):
-            return getattr(self, f'{run_method_name}_filter')(info)
-        return info

apps/accounts/automations/gather_account/host/posix/main.yml

@@ -1,61 +0,0 @@
-- hosts: demo
-  gather_facts: no
-  tasks:
-    - name: Get users
-      ansible.builtin.shell:
-        cmd: >
-          getent passwd | awk -F: '$7 !~ /(false|nologin|true|sync)$/' | grep -v '^$' | awk -F":" '{ print $1 }'
-      register: users
-
-    - name: Gather posix account last login
-      ansible.builtin.shell: |
-        for user in {{ users.stdout_lines | join(" ") }}; do
-          last -wi --time-format iso -n 1 ${user}  | awk '{ print $1,$3,$4, $NF }' | head -1 | awk 'NF'
-        done
-      register: last_login
-
-    - name: Get user password change date and expiry
-      ansible.builtin.shell: |
-        for user in {{ users.stdout_lines | join(" ") }}; do
-          k=$(getent shadow $user | awk -F: '{ print $3, $5 }')
-          echo "$user:$k"
-        done
-      register: passwd_date
-
-    - name: Get user groups
-      ansible.builtin.shell: |
-        for user in {{ users.stdout_lines | join(" ") }}; do
-          echo "$(groups $user)" | sed 's@ : @:@g'
-        done
-      register: user_groups
-
-    - name: Get sudoers
-      ansible.builtin.shell: |
-        for user in {{ users.stdout_lines | join(" ") }}; do
-          echo "$user: $(grep "^$user " /etc/sudoers | tr '\n' ';' || echo '')"
-        done
-      register: user_sudo
-
-    - name: Get authorized keys
-      ansible.builtin.shell: |
-        for user in {{ users.stdout_lines | join(" ") }}; do
-          home=$(getent passwd $user | cut -d: -f6)
-          echo -n "$user:"
-          if [ -f "${home}/.ssh/authorized_keys" ]; then
-            cat ${home}/.ssh/authorized_keys | tr '\n' ';'
-          fi
-          echo
-        done
-      register: user_authorized
-
-    - set_fact:
-        info:
-          users: "{{ users.stdout_lines }}"
-          last_login: "{{ last_login.stdout_lines }}"
-          user_groups: "{{ user_groups.stdout_lines }}"
-          user_sudo: "{{ user_sudo.stdout_lines }}"
-          user_authorized: "{{ user_authorized.stdout_lines }}"
-          passwd_date: "{{ passwd_date.stdout_lines }}"
-
-    - debug:
-        var: info

apps/accounts/automations/gather_account/host/windows/main.yml

@@ -1,33 +0,0 @@
-- hosts: demo
-  gather_facts: no
-  tasks:
-    - name: Run net user command to get all users
-      win_shell: net user
-      register: user_list_output
-      failed_when: false
-
-    - name: Parse all users from net user command
-      set_fact:
-        all_users: >-
-          {%- set users = [] -%}
-          {%- for line in user_list_output.stdout_lines -%}
-            {%- if loop.index > 4 and line.strip() != "" and not line.startswith("The command completed") -%}
-              {%- for user in line.split() -%}
-                {%- set _ = users.append(user) -%}
-              {%- endfor -%}
-            {%- endif -%}
-          {%- endfor -%}
-          {{ users }}
-
-    - name: Run net user command for each user to get details
-      win_shell: net user {{ item }}
-      loop: "{{ all_users }}"
-      register: user_details
-      ignore_errors: yes
-
-    - set_fact:
-        info:
-          user_details: "{{ user_details.results }}"
-
-    - debug:
-        var: info

apps/accounts/automations/gather_account/host/windows_ad/main.yml

@@ -1,74 +0,0 @@
-- hosts: demo
-  gather_facts: no
-  tasks:
-    - name: Import ActiveDirectory module
-      win_shell: Import-Module ActiveDirectory
-      args:
-        warn: false
-
-    - name: Get the SamAccountName list of all AD users
-      win_shell: |
-        Import-Module ActiveDirectory
-        Get-ADUser -Filter * | Select-Object -ExpandProperty SamAccountName
-      register: ad_user_list
-
-    - name: Set the all_users variable
-      set_fact:
-        all_users: "{{ ad_user_list.stdout_lines }}"
-
-    - name: Get detailed information for each user
-      win_shell: |
-        Import-Module ActiveDirectory
-    
-        $user = Get-ADUser -Identity {{ item }} -Properties Name, SamAccountName, Enabled, LastLogonDate, PasswordLastSet, msDS-UserPasswordExpiryTimeComputed, MemberOf
-    
-        $globalGroups = @()
-        if ($user.MemberOf) {
-          $globalGroups = $user.MemberOf | ForEach-Object {
-            try {
-              $group = Get-ADGroup $_ -ErrorAction Stop
-              if ($group.GroupScope -eq 'Global') { $group.Name }
-            } catch {
-            }
-          }
-        }
-    
-        $passwordExpiry = $null
-        $expiryRaw = $user.'msDS-UserPasswordExpiryTimeComputed'
-        if ($expiryRaw) {
-          try {
-            $passwordExpiry = [datetime]::FromFileTime($expiryRaw)
-          } catch {
-            $passwordExpiry = $null
-          }
-        }
-    
-        $output = [PSCustomObject]@{
-          Name                    = $user.Name
-          SamAccountName          = $user.SamAccountName
-          Enabled                 = $user.Enabled
-          LastLogonDate           = if ($user.LastLogonDate) { $user.LastLogonDate.ToString("yyyy-MM-dd HH:mm:ss") } else { $null }
-          PasswordLastSet         = if ($user.PasswordLastSet) { $user.PasswordLastSet.ToString("yyyy-MM-dd HH:mm:ss") } else { $null }
-          PasswordExpires         = if ($passwordExpiry) { $passwordExpiry.ToString("yyyy-MM-dd HH:mm:ss") } else { $null }
-          GlobalGroupMemberships  = $globalGroups
-        }
-    
-        $output | ConvertTo-Json -Depth 3
-      loop: "{{ all_users }}"
-      register: ad_user_details
-      ignore_errors: yes
-
-
-    - set_fact:
-        info:
-          user_details: >-
-            {{
-              ad_user_details.results
-              | selectattr('rc', 'equalto', 0)
-              | map(attribute='stdout')
-              | select('truthy')
-              | map('from_json')
-            }}
-
-    - debug:
-        var: info

apps/accounts/automations/gather_account/host/windows_ad/manifest.yml

@@ -1,15 +0,0 @@
-id: gather_accounts_windows_ad
-name: "{{ 'Windows account gather' | trans }}"
-version: 1
-method: gather_accounts
-category:
-  - ds
-
-type:
-  - windows_ad
-
-i18n:
-  Windows account gather:
-    zh: 使用命令 Get-ADUser 收集 Windows 账号
-    ja: コマンド Get-ADUser を使用して Windows アカウントを収集する
-    en: Using command Get-ADUser to gather accounts

apps/accounts/automations/gather_account/manager.py

@@ -1,409 +0,0 @@
-import time
-from collections import defaultdict
-
-from django.utils import timezone
-
-from accounts.const import AutomationTypes
-from accounts.models import GatheredAccount, Account, AccountRisk, RiskChoice
-from common.const import ConfirmOrIgnore
-from common.decorators import bulk_create_decorator, bulk_update_decorator
-from common.utils import get_logger
-from common.utils.strings import get_text_diff
-from orgs.utils import tmp_to_org
-from .filter import GatherAccountsFilter
-from ..base.manager import AccountBasePlaybookManager
-
-logger = get_logger(__name__)
-
-risk_items = [
-    "authorized_keys",
-    "sudoers",
-    "groups",
-]
-common_risk_items = [
-    "address_last_login",
-    "date_last_login",
-    "date_password_change",
-    "date_password_expired",
-    "detail"
-]
-diff_items = risk_items + common_risk_items
-
-
-def format_datetime(value):
-    if isinstance(value, timezone.datetime):
-        return value.strftime("%Y-%m-%d %H:%M:%S")
-    return value
-
-
-def get_items_diff(ori_account, d):
-    if hasattr(ori_account, "_diff"):
-        return ori_account._diff
-
-    diff = {}
-    for item in diff_items:
-        get_item_diff(item, ori_account, d, diff)
-    ori_account._diff = diff
-    return diff
-
-
-def get_item_diff(item, ori_account, d, diff):
-    detail = getattr(ori_account, 'detail', {})
-    new_detail = d.get('detail', {})
-    ori = getattr(ori_account, item, None) or detail.get(item)
-    new = d.get(item, "") or new_detail.get(item)
-    if not ori and not new:
-        return
-
-    ori = format_datetime(ori)
-    new = format_datetime(new)
-
-    if new != ori:
-        diff[item] = get_text_diff(str(ori), str(new))
-
-
-class AnalyseAccountRisk:
-    long_time = timezone.timedelta(days=90)
-    datetime_check_items = [
-        {"field": "date_last_login", "risk": "long_time_no_login", "delta": long_time},
-        {
-            "field": "date_password_change",
-            "risk": RiskChoice.long_time_password,
-            "delta": long_time,
-        },
-        {
-            "field": "date_password_expired",
-            "risk": "password_expired",
-            "delta": timezone.timedelta(seconds=1),
-        },
-    ]
-
-    def __init__(self, check_risk=True):
-        self.check_risk = check_risk
-        self.now = timezone.now()
-        self.pending_add_risks = []
-
-    def _analyse_item_changed(self, ori_ga, d):
-        diff = get_items_diff(ori_ga, d)
-        if not diff:
-            return
-
-        risks = []
-        for k, v in diff.items():
-            if k not in risk_items:
-                continue
-            risks.append(
-                dict(
-                    asset_id=str(ori_ga.asset_id),
-                    username=ori_ga.username,
-                    gathered_account=ori_ga,
-                    risk=k + "_changed",
-                    detail={"diff": v},
-                )
-            )
-        self.save_or_update_risks(risks)
-
-    def _analyse_datetime_changed(self, ori_account, d, asset, username):
-        basic = {"asset_id": str(asset.id), "username": username}
-
-        risks = []
-        for item in self.datetime_check_items:
-            field = item["field"]
-            risk = item["risk"]
-            delta = item["delta"]
-
-            date = d.get(field)
-            if not date:
-                continue
-
-            # 服务器收集的时间和数据库时间一致，不进行比较，无法检测风险 不太对，先注释
-            # pre_date = ori_account and getattr(ori_account, field)
-            # if pre_date == date:
-            #     continue
-
-            if date and date < timezone.now() - delta:
-                risks.append(
-                    dict(**basic, risk=risk, detail={"date": date.isoformat()})
-                )
-
-        self.save_or_update_risks(risks)
-
-    def save_or_update_risks(self, risks):
-        # 提前取出来，避免每次都查数据库
-        asset_ids = {r["asset_id"] for r in risks}
-        assets_risks = AccountRisk.objects.filter(asset_id__in=asset_ids)
-        assets_risks = {f"{r.asset_id}_{r.username}_{r.risk}": r for r in assets_risks}
-
-        for d in risks:
-            detail = d.pop("detail", {})
-            detail["datetime"] = self.now.isoformat()
-            key = f"{d['asset_id']}_{d['username']}_{d['risk']}"
-            found = assets_risks.get(key)
-
-            if not found:
-                self._create_risk(dict(**d, details=[detail]))
-                continue
-
-            found.details.append(detail)
-            self._update_risk(found)
-
-    @bulk_create_decorator(AccountRisk)
-    def _create_risk(self, data):
-        return AccountRisk(**data)
-
-    @bulk_update_decorator(AccountRisk, update_fields=["details"])
-    def _update_risk(self, account):
-        return account
-
-    def lost_accounts(self, asset, lost_users):
-        if not self.check_risk:
-            return
-        for user in lost_users:
-            self._create_risk(
-                dict(
-                    asset_id=str(asset.id),
-                    username=user,
-                    risk=RiskChoice.account_deleted,
-                    details=[{"datetime": self.now.isoformat()}],
-                )
-            )
-
-    def analyse_risk(self, asset, ga, d, sys_found):
-        if not self.check_risk:
-            return
-
-        if ga:
-            self._analyse_item_changed(ga, d)
-        if not sys_found:
-            basic = {"asset": asset, "username": d["username"], 'gathered_account': ga}
-            self._create_risk(
-                dict(
-                    **basic,
-                    risk=RiskChoice.new_found,
-                    details=[{"datetime": self.now.isoformat()}],
-                )
-            )
-        self._analyse_datetime_changed(ga, d, asset, d["username"])
-
-
-class GatherAccountsManager(AccountBasePlaybookManager):
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self.host_asset_mapper = {}
-        self.asset_account_info = {}
-        self.asset_usernames_mapper = defaultdict(set)
-        self.ori_asset_usernames = defaultdict(set)
-        self.ori_gathered_usernames = defaultdict(set)
-        self.ori_gathered_accounts_mapper = dict()
-        self.is_sync_account = self.execution.snapshot.get("is_sync_account")
-        self.check_risk = self.execution.snapshot.get("check_risk", False)
-
-    @classmethod
-    def method_type(cls):
-        return AutomationTypes.gather_accounts
-
-    def host_callback(self, host, asset=None, **kwargs):
-        super().host_callback(host, asset=asset, **kwargs)
-        self.host_asset_mapper[host["name"]] = asset
-        return host
-
-    def _filter_success_result(self, tp, result):
-        result = GatherAccountsFilter(tp).run(self.method_id_meta_mapper, result)
-        return result
-
-    @staticmethod
-    def _get_nested_info(data, *keys):
-        for key in keys:
-            data = data.get(key, {})
-            if not data:
-                break
-        return data
-
-    def _collect_asset_account_info(self, asset, info):
-        result = self._filter_success_result(asset.type, info)
-        accounts = []
-
-        for username, info in result.items():
-            self.asset_usernames_mapper[str(asset.id)].add(username)
-
-            d = {"asset": asset, "username": username, "remote_present": True, **info}
-            accounts.append(d)
-        self.asset_account_info[asset] = accounts
-
-    def on_host_success(self, host, result):
-        super().on_host_success(host, result)
-        info = self._get_nested_info(result, "debug", "res", "info")
-        asset = self.host_asset_mapper.get(host)
-
-        if asset and info:
-            self._collect_asset_account_info(asset, info)
-        else:
-            print(f"\033[31m Not found {host} info \033[0m\n")
-
-    def prefetch_origin_account_usernames(self):
-        """
-        提起查出来，避免每次 sql 查询
-        :return:
-        """
-        assets = self.asset_usernames_mapper.keys()
-        accounts = Account.objects.filter(asset__in=assets).values_list(
-            "asset", "username"
-        )
-
-        for asset_id, username in accounts:
-            self.ori_asset_usernames[str(asset_id)].add(username)
-
-        ga_accounts = GatheredAccount.objects.filter(asset__in=assets)
-        for account in ga_accounts:
-            self.ori_gathered_usernames[str(account.asset_id)].add(account.username)
-            key = "{}_{}".format(account.asset_id, account.username)
-            self.ori_gathered_accounts_mapper[key] = account
-
-    def update_gather_accounts_status(self, asset):
-        """
-        远端账号，收集中的账号，vault 中的账号。
-        要根据账号新增见啥，标识 收集账号的状态, 让管理员关注
-
-        远端账号 -> 收集账号 -> 特权账号
-        """
-        remote_users = self.asset_usernames_mapper[str(asset.id)]
-        ori_users = self.ori_asset_usernames[str(asset.id)]
-        ori_ga_users = self.ori_gathered_usernames[str(asset.id)]
-
-        queryset = GatheredAccount.objects.filter(asset=asset).exclude(
-            status=ConfirmOrIgnore.ignored
-        )
-
-        # 远端账号 比 收集账号多的
-        # 新增创建，不用处理状态
-        new_found_users = remote_users - ori_ga_users
-        if new_found_users:
-            self.summary["new_accounts"] += len(new_found_users)
-            for username in new_found_users:
-                self.result["new_accounts"].append(
-                    {
-                        "asset": str(asset),
-                        "username": username,
-                    }
-                )
-
-        # 远端上 比 收集账号少的
-        # 标识 remote_present=False, 标记为待处理
-        # 远端资产上不存在的，标识为待处理，需要管理员介入
-        lost_users = ori_ga_users - remote_users
-        if lost_users:
-            queryset.filter(username__in=lost_users).update(
-                status=ConfirmOrIgnore.pending, remote_present=False
-            )
-            self.summary["lost_accounts"] += len(lost_users)
-            for username in lost_users:
-                self.result["lost_accounts"].append(
-                    {
-                        "asset": str(asset),
-                        "username": username,
-                    }
-                )
-            risk_analyser = AnalyseAccountRisk(self.check_risk)
-            risk_analyser.lost_accounts(asset, lost_users)
-
-        # 收集的账号 比 账号列表多的, 有可能是账号中删掉了, 但这时候状态已经是 confirm 了
-        # 标识状态为 待处理, 让管理员去确认
-        ga_added_users = ori_ga_users - ori_users
-        if ga_added_users:
-            queryset.filter(username__in=ga_added_users).update(status=ConfirmOrIgnore.pending)
-
-        # 收集的账号 比 账号列表少的
-        # 这个好像不不用对比，原始情况就这样
-
-        # 远端账号 比 账号列表少的
-        # 创建收集账号，标识 remote_present=False, 状态待处理
-
-        # 远端账号 比 账号列表多的
-        # 正常情况, 不用处理，因为远端账号会创建到收集账号，收集账号再去对比
-
-        # 不过这个好像也处理一下 status，因为已存在，这是状态应该是确认
-        (
-            queryset.filter(username__in=ori_users)
-            .exclude(status=ConfirmOrIgnore.confirmed)
-            .update(status=ConfirmOrIgnore.confirmed)
-        )
-
-        # 远端存在的账号，标识为已存在
-        (
-            queryset.filter(username__in=remote_users, remote_present=False).update(
-                remote_present=True
-            )
-        )
-
-        # 资产上没有的，标识为为存在
-        (
-            queryset.exclude(username__in=ori_users)
-            .filter(present=True)
-            .update(present=False)
-        )
-        (
-            queryset.filter(username__in=ori_users)
-            .filter(present=False)
-            .update(present=True)
-        )
-
-    @bulk_create_decorator(GatheredAccount)
-    def create_gathered_account(self, d):
-        ga = GatheredAccount()
-        for k, v in d.items():
-            setattr(ga, k, v)
-
-        return ga
-
-    @bulk_update_decorator(GatheredAccount, update_fields=common_risk_items)
-    def update_gathered_account(self, ori_account, d):
-        diff = get_items_diff(ori_account, d)
-        if not diff:
-            return
-        for k in diff:
-            if k not in common_risk_items:
-                continue
-            v = d.get(k)
-            setattr(ori_account, k, v)
-        return ori_account
-
-    def do_run(self, *args, **kwargs):
-        super().do_run(*args, **kwargs)
-        self.prefetch_origin_account_usernames()
-        risk_analyser = AnalyseAccountRisk(self.check_risk)
-
-        for asset, accounts_data in self.asset_account_info.items():
-            ori_users = self.ori_asset_usernames[str(asset.id)]
-            need_analyser_gather_account = []
-            with tmp_to_org(asset.org_id):
-                for d in accounts_data:
-                    username = d["username"]
-                    ori_account = self.ori_gathered_accounts_mapper.get(
-                        "{}_{}".format(asset.id, username)
-                    )
-                    if not ori_account:
-                        ga = self.create_gathered_account(d)
-                    else:
-                        ga = ori_account
-                        self.update_gathered_account(ori_account, d)
-                    ori_found = username in ori_users
-                    need_analyser_gather_account.append((asset, ga, d, ori_found))
-                self.create_gathered_account.finish()
-                self.update_gathered_account.finish()
-                for analysis_data in need_analyser_gather_account:
-                    risk_analyser.analyse_risk(*analysis_data)
-                self.update_gather_accounts_status(asset)
-                if not self.is_sync_account:
-                    continue
-                gathered_accounts = GatheredAccount.objects.filter(asset=asset)
-                GatheredAccount.sync_accounts(gathered_accounts)
-                GatheredAccount.objects.filter(
-                    asset=asset, username__in=ori_users, present=False
-                ).update(
-                    present=True
-                )
-        # 因为有 bulk create, bulk update, 所以这里需要 sleep 一下，等待数据同步
-        time.sleep(0.5)
-
-    def get_report_template(self):
-        return "accounts/gather_account_report.html"

apps/accounts/automations/gather_accounts/database/mongodb/main.yml

@@ -1,7 +1,7 @@
 - hosts: mongodb
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Get info
@@ -15,7 +15,7 @@
         ssl_ca_certs: "{{ jms_asset.secret_info.ca_cert | default('') }}"
         ssl_certfile: "{{ jms_asset.secret_info.client_key | default('') }}"
         connection_options:
-          - tlsAllowInvalidHostnames: "{{ jms_asset.spec_info.allow_invalid_cert }}"
+          - tlsAllowInvalidHostnames: "{{ jms_asset.spec_info.allow_invalid_cert}}"
         filter: users
       register: db_info
 

apps/accounts/automations/gather_accounts/database/mysql/main.yml

@@ -1,11 +1,8 @@
 - hosts: mysql
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
     check_ssl: "{{ jms_asset.spec_info.use_ssl and not jms_asset.spec_info.allow_invalid_cert }}"
-    ca_cert: "{{ jms_asset.secret_info.ca_cert | default('') }}"
-    ssl_cert: "{{ jms_asset.secret_info.client_cert | default('') }}"
-    ssl_key: "{{ jms_asset.secret_info.client_key | default('') }}"
 
   tasks:
     - name: Get info
@@ -15,9 +12,9 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         filter: users
       register: db_info
 

apps/accounts/automations/gather_accounts/database/oracle/main.yml

@@ -1,7 +1,7 @@
 - hosts: oralce
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Get info

apps/accounts/automations/gather_accounts/database/postgresql/main.yml

@@ -0,0 +1,22 @@
+- hosts: postgresql
+  gather_facts: no
+  vars:
+    ansible_python_interpreter: /opt/py3/bin/python
+
+  tasks:
+    - name: Get info
+      community.postgresql.postgresql_info:
+        login_user: "{{ jms_account.username }}"
+        login_password: "{{ jms_account.secret }}"
+        login_host: "{{ jms_asset.address }}"
+        login_port: "{{ jms_asset.port }}"
+        login_db: "{{ jms_asset.spec_info.db_name }}"
+        filter: "roles"
+      register: db_info
+
+    - name: Define info by set_fact
+      set_fact:
+        info: "{{ db_info.roles }}"
+
+    - debug:
+        var: info

apps/accounts/automations/gather_accounts/filter.py

@@ -0,0 +1,74 @@
+import re
+
+from django.utils import timezone
+
+__all__ = ['GatherAccountsFilter']
+
+
+# TODO 后期会挪到playbook中
+class GatherAccountsFilter:
+
+    def __init__(self, tp):
+        self.tp = tp
+
+    @staticmethod
+    def mysql_filter(info):
+        result = {}
+        for _, user_dict in info.items():
+            for username, _ in user_dict.items():
+                if len(username.split('.')) == 1:
+                    result[username] = {}
+        return result
+
+    @staticmethod
+    def postgresql_filter(info):
+        result = {}
+        for username in info:
+            result[username] = {}
+        return result
+
+    @staticmethod
+    def posix_filter(info):
+        username_pattern = re.compile(r'^(\S+)')
+        ip_pattern = re.compile(r'(\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})')
+        login_time_pattern = re.compile(r'\w{3} \d{2} \d{2}:\d{2}:\d{2} \d{4}')
+        result = {}
+        for line in info:
+            usernames = username_pattern.findall(line)
+            username = ''.join(usernames)
+            if username:
+                result[username] = {}
+            else:
+                continue
+            ip_addrs = ip_pattern.findall(line)
+            ip_addr = ''.join(ip_addrs)
+            if ip_addr:
+                result[username].update({'address': ip_addr})
+            login_times = login_time_pattern.findall(line)
+            if login_times:
+                date = timezone.datetime.strptime(f'{login_times[0]} +0800', '%b %d %H:%M:%S %Y %z')
+                result[username].update({'date': date})
+        return result
+
+    @staticmethod
+    def windows_filter(info):
+        info = info[4:-2]
+        result = {}
+        for i in info:
+            for username in i.split():
+                result[username] = {}
+        return result
+
+    def run(self, method_id_meta_mapper, info):
+        run_method_name = None
+        for k, v in method_id_meta_mapper.items():
+            if self.tp not in v['type']:
+                continue
+            run_method_name = k.replace(f'{v["method"]}_', '')
+
+        if not run_method_name:
+            return info
+
+        if hasattr(self, f'{run_method_name}_filter'):
+            return getattr(self, f'{run_method_name}_filter')(info)
+        return info

apps/accounts/automations/gather_accounts/host/posix/main.yml

@@ -0,0 +1,21 @@
+- hosts: demo
+  gather_facts: no
+  tasks:
+    - name: Gather posix account
+      ansible.builtin.shell:
+        cmd: >
+          users=$(getent passwd | grep -v nologin | grep -v shutdown | awk -F":" '{ print $1 }');for i in $users;
+          do k=$(last -w -F $i -1 | head -1 | grep -v ^$ | awk '{ print $0 }')
+            if [ -n "$k" ]; then
+              echo $k
+            else
+              echo $i
+            fi;done
+      register: result
+
+    - name: Define info by set_fact
+      ansible.builtin.set_fact:
+        info: "{{ result.stdout_lines }}"
+
+    - debug:
+        var: info

apps/accounts/automations/gather_accounts/host/windows/main.yml

@@ -0,0 +1,14 @@
+- hosts: demo
+  gather_facts: no
+  tasks:
+    - name: Gather windows account
+      ansible.builtin.win_shell: net user
+      register: result
+      ignore_errors: true
+
+    - name: Define info by set_fact
+      ansible.builtin.set_fact:
+        info: "{{ result.stdout_lines }}"
+
+    - debug:
+        var: info

apps/accounts/automations/gather_accounts/host/windows/manifest.yml

@@ -2,13 +2,10 @@ id: gather_accounts_windows
 name: "{{ 'Windows account gather' | trans }}"
 version: 1
 method: gather_accounts
-category:
-  - host
-
+category: host
 type:
   - windows
 
-
 i18n:
   Windows account gather:
     zh: 使用命令 net user 收集 Windows 账号

apps/accounts/automations/gather_accounts/manager.py

@@ -0,0 +1,139 @@
+from collections import defaultdict
+
+from accounts.const import AutomationTypes
+from accounts.models import GatheredAccount
+from assets.models import Asset
+from common.utils import get_logger
+from orgs.utils import tmp_to_org
+from users.models import User
+from .filter import GatherAccountsFilter
+from ..base.manager import AccountBasePlaybookManager
+from ...notifications import GatherAccountChangeMsg
+
+logger = get_logger(__name__)
+
+
+class GatherAccountsManager(AccountBasePlaybookManager):
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.host_asset_mapper = {}
+        self.asset_account_info = {}
+
+        self.asset_username_mapper = defaultdict(set)
+        self.is_sync_account = self.execution.snapshot.get('is_sync_account')
+
+    @classmethod
+    def method_type(cls):
+        return AutomationTypes.gather_accounts
+
+    def host_callback(self, host, asset=None, **kwargs):
+        super().host_callback(host, asset=asset, **kwargs)
+        self.host_asset_mapper[host['name']] = asset
+        return host
+
+    def filter_success_result(self, tp, result):
+        result = GatherAccountsFilter(tp).run(self.method_id_meta_mapper, result)
+        return result
+
+    def generate_data(self, asset, result):
+        data = []
+        for username, info in result.items():
+            self.asset_username_mapper[str(asset.id)].add(username)
+            d = {'asset': asset, 'username': username, 'present': True}
+            if info.get('date'):
+                d['date_last_login'] = info['date']
+            if info.get('address'):
+                d['address_last_login'] = info['address'][:32]
+            data.append(d)
+        return data
+
+    def collect_asset_account_info(self, asset, result):
+        data = self.generate_data(asset, result)
+        self.asset_account_info[asset] = data
+
+    @staticmethod
+    def get_nested_info(data, *keys):
+        for key in keys:
+            data = data.get(key, {})
+            if not data:
+                break
+        return data
+
+    def on_host_success(self, host, result):
+        info = self.get_nested_info(result, 'debug', 'res', 'info')
+        asset = self.host_asset_mapper.get(host)
+        if asset and info:
+            result = self.filter_success_result(asset.type, info)
+            self.collect_asset_account_info(asset, result)
+        else:
+            print(f'\033[31m Not found {host} info \033[0m\n')
+
+    def update_or_create_accounts(self):
+        for asset, data in self.asset_account_info.items():
+            with tmp_to_org(asset.org_id):
+                gathered_accounts = []
+                GatheredAccount.objects.filter(asset=asset, present=True).update(present=False)
+                for d in data:
+                    username = d['username']
+                    gathered_account, __ = GatheredAccount.objects.update_or_create(
+                        defaults=d, asset=asset, username=username,
+                    )
+                    gathered_accounts.append(gathered_account)
+                if not self.is_sync_account:
+                    continue
+                GatheredAccount.sync_accounts(gathered_accounts)
+
+    def run(self, *args, **kwargs):
+        super().run(*args, **kwargs)
+        users, change_info = self.generate_send_users_and_change_info()
+        self.update_or_create_accounts()
+        self.send_email_if_need(users, change_info)
+
+    def generate_send_users_and_change_info(self):
+        recipients = self.execution.recipients
+        if not self.asset_username_mapper or not recipients:
+            return None, None
+
+        users = User.objects.filter(id__in=recipients)
+        if not users:
+            return users, None
+
+        asset_ids = self.asset_username_mapper.keys()
+        assets = Asset.objects.filter(id__in=asset_ids)
+        gather_accounts = GatheredAccount.objects.filter(asset_id__in=asset_ids, present=True)
+        asset_id_map = {str(asset.id): asset for asset in assets}
+        asset_id_username = list(assets.values_list('id', 'accounts__username'))
+        asset_id_username.extend(list(gather_accounts.values_list('asset_id', 'username')))
+
+        system_asset_username_mapper = defaultdict(set)
+        for asset_id, username in asset_id_username:
+            system_asset_username_mapper[str(asset_id)].add(username)
+
+        change_info = {}
+        for asset_id, usernames in self.asset_username_mapper.items():
+            system_usernames = system_asset_username_mapper.get(asset_id)
+
+            if not system_usernames:
+                continue
+
+            add_usernames = usernames - system_usernames
+            remove_usernames = system_usernames - usernames
+            k = f'{asset_id_map[asset_id]}[{asset_id}]'
+
+            if not add_usernames and not remove_usernames:
+                continue
+
+            change_info[k] = {
+                'add_usernames': ', '.join(add_usernames),
+                'remove_usernames': ', '.join(remove_usernames),
+            }
+
+        return users, change_info
+
+    @staticmethod
+    def send_email_if_need(users, change_info):
+        if not users or not change_info:
+            return
+
+        for user in users:
+            GatherAccountChangeMsg(user, change_info).publish_async()

apps/accounts/automations/push_account/custom/ssh/main.yml

@@ -1,62 +0,0 @@
-- hosts: custom
-  gather_facts: no
-  vars:
-    ansible_connection: local
-    ansible_become: false
-
-  tasks:
-    - name: Test privileged account (paramiko)
-      ssh_ping:
-        login_host: "{{ jms_asset.address }}"
-        login_port: "{{ jms_asset.port }}"
-        login_user: "{{ jms_account.username }}"
-        login_password: "{{ jms_account.secret }}"
-        login_secret_type: "{{ jms_account.secret_type }}"
-        login_private_key_path: "{{ jms_account.private_key_path }}"
-        become: "{{ jms_custom_become | default(False) }}"
-        become_method: "{{ jms_custom_become_method | default('su') }}"
-        become_user: "{{ jms_custom_become_user | default('') }}"
-        become_password: "{{ jms_custom_become_password | default('') }}"
-        become_private_key_path: "{{ jms_custom_become_private_key_path | default(None) }}"
-        old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
-      register: ping_info
-      delegate_to: localhost
-
-    - name: Change asset password (paramiko)
-      custom_command:
-        login_user: "{{ jms_account.username }}"
-        login_password: "{{ jms_account.secret }}"
-        login_host: "{{ jms_asset.address }}"
-        login_port: "{{ jms_asset.port }}"
-        login_secret_type: "{{ jms_account.secret_type }}"
-        login_private_key_path: "{{ jms_account.private_key_path }}"
-        become: "{{ jms_custom_become | default(False) }}"
-        become_method: "{{ jms_custom_become_method | default('su') }}"
-        become_user: "{{ jms_custom_become_user | default('') }}"
-        become_password: "{{ jms_custom_become_password | default('') }}"
-        become_private_key_path: "{{ jms_custom_become_private_key_path | default(None) }}"
-        name: "{{ account.username }}"
-        password: "{{ account.secret }}"
-        commands: "{{ params.commands }}"
-        first_conn_delay_time: "{{ first_conn_delay_time | default(0.5) }}"
-      ignore_errors: true
-      when: ping_info is succeeded and check_conn_after_change
-      register: change_info
-      delegate_to: localhost
-
-    - name: Verify password (paramiko)
-      ssh_ping:
-        login_user: "{{ account.username }}"
-        login_password: "{{ account.secret }}"
-        login_host: "{{ jms_asset.address }}"
-        login_port: "{{ jms_asset.port }}"
-        become: "{{ account.become.ansible_become | default(False) }}"
-        become_method: su
-        become_user: "{{ account.become.ansible_user | default('') }}"
-        become_password: "{{ account.become.ansible_password | default('') }}"
-        become_private_key_path: "{{ account.become.ansible_ssh_private_key_file | default(None) }}"
-        old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
-      delegate_to: localhost
-      when: check_conn_after_change

apps/accounts/automations/push_account/custom/ssh/manifest.yml

@@ -1,32 +0,0 @@
-id: push_account_by_ssh
-name: "{{ 'SSH account push' | trans }}"
-category:
-  - device
-  - host
-type:
-  - all
-method: push_account
-protocol: ssh
-priority: 50
-params:
-  - name: commands
-    type: list
-    label: "{{ 'Params commands label' | trans }}"
-    default: [ '' ]
-    help_text: "{{ 'Params commands help text' | trans }}"
-
-i18n:
-  SSH account push:
-    zh: '使用 SSH 命令行自定义推送'
-    ja: 'SSHコマンドラインを使用してプッシュをカスタマイズする'
-    en: 'Custom push using SSH command line'
-
-  Params commands help text:
-    zh: '自定义命令中如需包含账号的 账号、密码、SSH 连接的用户密码 字段，<br />请使用 &#123;username&#125;、&#123;password&#125;、&#123;login_password&#125;格式，执行任务时会进行替换 。<br />比如针对 Cisco 主机进行改密，一般需要配置五条命令：<br />1. enable<br />2. &#123;login_password&#125;<br />3. configure terminal<br />4. username &#123;username&#125; privilege 0 password &#123;password&#125; <br />5. end'
-    ja: 'カスタム コマンドに SSH 接続用のアカウント番号、パスワード、ユーザー パスワード フィールドを含める必要がある場合は、<br />&#123;ユーザー名&#125;、&#123;パスワード&#125;、&#123;login_password& を使用してください。 # 125; 形式。タスクの実行時に置き換えられます。 <br />たとえば、Cisco ホストのパスワードを変更するには、通常、次の 5 つのコマンドを設定する必要があります:<br />1.enable<br />2.&#123;login_password&#125;<br />3 .ターミナルの設定<br / >4. ユーザー名 &#123;ユーザー名&#125; 権限 0 パスワード &#123;パスワード&#125; <br />5. 終了'
-    en: 'If the custom command needs to include the account number, password, and user password field for SSH connection,<br />Please use &#123;username&#125;, &#123;password&#125;, &#123;login_password&# 125; format, which will be replaced when executing the task. <br />For example, to change the password of a Cisco host, you generally need to configure five commands:<br />1. enable<br />2. &#123;login_password&#125;<br />3. configure terminal<br / >4. username &#123;username&#125; privilege 0 password &#123;password&#125; <br />5. end'
-
-  Params commands label:
-    zh: '自定义命令'
-    ja: 'カスタムコマンド'
-    en: 'Custom command'

apps/accounts/automations/push_account/database/mongodb/main.yml

@@ -1,7 +1,7 @@
 - hosts: mongodb
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test MongoDB connection
@@ -53,4 +53,3 @@
         ssl_certfile: "{{ jms_asset.secret_info.client_key | default('') }}"
         connection_options:
           - tlsAllowInvalidHostnames: "{{ jms_asset.spec_info.allow_invalid_cert}}"
-      when: check_conn_after_change

apps/accounts/automations/push_account/database/mysql/main.yml

@@ -1,12 +1,9 @@
 - hosts: mysql
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
     db_name: "{{ jms_asset.spec_info.db_name }}"
     check_ssl: "{{ jms_asset.spec_info.use_ssl and not jms_asset.spec_info.allow_invalid_cert }}"
-    ca_cert: "{{ jms_asset.secret_info.ca_cert | default('') }}"
-    ssl_cert: "{{ jms_asset.secret_info.client_cert | default('') }}"
-    ssl_key: "{{ jms_asset.secret_info.client_key | default('') }}"
 
   tasks:
     - name: Test MySQL connection
@@ -16,9 +13,9 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         filter: version
       register: db_info
 
@@ -33,9 +30,9 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         name: "{{ account.username }}"
         password: "{{ account.secret }}"
         host: "%"
@@ -50,8 +47,7 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         check_hostname: "{{ check_ssl if check_ssl else omit }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        client_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        client_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
+        ca_cert: "{{ jms_asset.secret_info.ca_cert | default(omit) if check_ssl else omit }}"
+        client_cert: "{{ jms_asset.secret_info.client_cert | default(omit) if check_ssl else omit }}"
+        client_key: "{{ jms_asset.secret_info.client_key | default(omit) if check_ssl else omit }}"
         filter: version
-      when: check_conn_after_change

apps/accounts/automations/push_account/database/oracle/main.yml

@@ -1,7 +1,7 @@
 - hosts: oracle
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test Oracle connection
@@ -40,4 +40,3 @@
         login_port: "{{ jms_asset.port }}"
         login_database: "{{ jms_asset.spec_info.db_name }}"
         mode: "{{ account.mode }}"
-      when: check_conn_after_change

apps/accounts/automations/push_account/database/postgresql/main.yml

@@ -1,11 +1,7 @@
 - hosts: postgre
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
-    check_ssl: "{{ jms_asset.spec_info.use_ssl }}"
-    ca_cert: "{{ jms_asset.secret_info.ca_cert | default('') }}"
-    ssl_cert: "{{ jms_asset.secret_info.client_cert | default('') }}"
-    ssl_key: "{{ jms_asset.secret_info.client_key | default('') }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test PostgreSQL connection
@@ -15,10 +11,6 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         login_db: "{{ jms_asset.spec_info.db_name }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
       register: result
       failed_when: not result.is_available
 
@@ -36,10 +28,6 @@
         db: "{{ jms_asset.spec_info.db_name }}"
         name: "{{ account.username }}"
         password: "{{ account.secret }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
         role_attr_flags: LOGIN
       ignore_errors: true
       when: result is succeeded
@@ -52,13 +40,8 @@
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
         db: "{{ jms_asset.spec_info.db_name }}"
-        ca_cert: "{{ ca_cert if check_ssl and ca_cert | length > 0 else omit }}"
-        ssl_cert: "{{ ssl_cert if check_ssl and ssl_cert | length > 0 else omit }}"
-        ssl_key: "{{ ssl_key if check_ssl and ssl_key | length > 0 else omit }}"
-        ssl_mode: "{{ jms_asset.spec_info.pg_ssl_mode }}"
       when:
         - result is succeeded
         - change_info is succeeded
-        - check_conn_after_change
       register: result
       failed_when: not result.is_available

apps/accounts/automations/push_account/database/sqlserver/main.yml

@@ -1,7 +1,7 @@
 - hosts: sqlserver
   gather_facts: no
   vars:
-    ansible_python_interpreter: "{{ local_python_interpreter }}"
+    ansible_python_interpreter: /opt/py3/bin/python
 
   tasks:
     - name: Test SQLServer connection
@@ -66,4 +66,3 @@
         name: '{{ jms_asset.spec_info.db_name }}'
         script: |
           SELECT @@version
-      when: check_conn_after_change

apps/accounts/automations/push_account/host/aix/main.yml

@@ -9,20 +9,55 @@
         database: passwd
         key: "{{ account.username }}"
       register: user_info
-      failed_when: false
-      changed_when: false
+      ignore_errors: yes  # 忽略错误，如果用户不存在时不会导致playbook失败
 
     - name: "Add {{ account.username }} user"
       ansible.builtin.user:
         name: "{{ account.username }}"
-        uid: "{{ params.uid | int if params.uid | length > 0 else omit }}"
-        shell: "{{ params.shell if params.shell | length > 0 else omit }}"
-        home: "{{ params.home if params.home | length > 0 else '/home/' + account.username }}"
-        groups: "{{ params.groups if params.groups | length > 0 else omit }}"
-        append: "{{ true if params.groups | length > 0 else false }}"
+        shell: "{{ params.shell }}"
+        home: "{{ params.home | default('/home/' + account.username, true) }}"
+        groups: "{{ params.groups }}"
         expires: -1
         state: present
-      when: user_info.msg is defined
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} group"
+      ansible.builtin.group:
+        name: "{{ account.username }}"
+        state: present
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} user to group"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        groups: "{{ params.groups }}"
+      when:
+        - user_info.failed
+        - params.groups
+
+    - name: "Change {{ account.username }} password"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        password: "{{ account.secret | password_hash('des') }}"
+        update_password: always
+      ignore_errors: true
+      when: account.secret_type == "password"
+
+    - name: remove jumpserver ssh key
+      ansible.builtin.lineinfile:
+        dest: "{{ ssh_params.dest }}"
+        regexp: "{{ ssh_params.regexp }}"
+        state: absent
+      when:
+        - account.secret_type == "ssh_key"
+        - ssh_params.strategy == "set_jms"
+
+    - name: "Change {{ account.username }} SSH key"
+      ansible.builtin.authorized_key:
+        user: "{{ account.username }}"
+        key: "{{ account.secret }}"
+        exclusive: "{{ ssh_params.exclusive }}"
+      when: account.secret_type == "ssh_key"
 
     - name: "Set {{ account.username }} sudo setting"
       ansible.builtin.lineinfile:
@@ -32,61 +67,9 @@
         line: "{{ account.username + ' ALL=(ALL) NOPASSWD: ' + params.sudo }}"
         validate: visudo -cf %s
       when:
-        - user_info.msg is defined or params.modify_sudo
+        - user_info.failed
         - params.sudo
 
-    - name: "Change {{ account.username }} password"
-      ansible.builtin.user:
-        name: "{{ account.username }}"
-        password: "{{ account.secret | password_hash('des') }}"
-        update_password: always
-      ignore_errors: true
-      register: change_secret_result
-      when: account.secret_type == "password"
-
-    - name: "Get home directory for {{ account.username }}"
-      ansible.builtin.shell: "getent passwd {{ account.username }} | cut -d: -f6"
-      register: home_dir
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Check if home directory exists for {{ account.username }}"
-      ansible.builtin.stat:
-        path: "{{ home_dir.stdout.strip() }}"
-      register: home_dir_stat
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Ensure {{ account.username }} home directory exists"
-      ansible.builtin.file:
-        path: "{{ home_dir.stdout.strip() }}"
-        state: directory
-        owner: "{{ account.username }}"
-        group: "{{ account.username }}"
-        mode: '0750'
-      when:
-        - account.secret_type == "ssh_key"
-        - home_dir_stat.stat.exists == false
-      ignore_errors: yes
-
-    - name: Remove jumpserver ssh key
-      ansible.builtin.lineinfile:
-        dest: "{{ home_dir.stdout.strip() }}/.ssh/authorized_keys"
-        regexp: "{{ ssh_params.regexp }}"
-        state: absent
-      when:
-        - account.secret_type == "ssh_key"
-        - ssh_params.strategy == "set_jms"
-      ignore_errors: yes
-
-    - name: "Change {{ account.username }} SSH key"
-      ansible.builtin.authorized_key:
-        user: "{{ account.username }}"
-        key: "{{ account.secret }}"
-        exclusive: "{{ ssh_params.exclusive }}"
-      register: change_secret_result
-      when: account.secret_type == "ssh_key"
-
     - name: Refresh connection
       ansible.builtin.meta: reset_connection
 
@@ -96,16 +79,14 @@
         login_password: "{{ account.secret }}"
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         become: "{{ account.become.ansible_become | default(False) }}"
         become_method: su
         become_user: "{{ account.become.ansible_user | default('') }}"
         become_password: "{{ account.become.ansible_password | default('') }}"
         become_private_key_path: "{{ account.become.ansible_ssh_private_key_file | default(None) }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "password"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "password"
       delegate_to: localhost
 
     - name: "Verify {{ account.username }} SSH KEY (paramiko)"
@@ -114,10 +95,8 @@
         login_port: "{{ jms_asset.port }}"
         login_user: "{{ account.username }}"
         login_private_key_path: "{{ account.private_key_path  }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "ssh_key"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "ssh_key"
       delegate_to: localhost
 

apps/accounts/automations/push_account/host/aix/manifest.yml

@@ -5,12 +5,6 @@ type:
   - AIX
 method: push_account
 params:
-  - name: modify_sudo
-    type: bool
-    label: "{{ 'Modify sudo label' | trans }}"
-    default: False
-    help_text: "{{ 'Modify params sudo help text' | trans }}"
-
   - name: sudo
     type: str
     label: 'Sudo'
@@ -34,23 +28,12 @@ params:
     default: ''
     help_text: "{{ 'Params groups help text' | trans }}"
 
-  - name: uid
-    type: str
-    label: "{{ 'Params uid label' | trans }}"
-    default: ''
-    help_text: "{{ 'Params uid help text' | trans }}"
-
 i18n:
   Aix account push:
     zh: '使用 Ansible 模块 user 执行 Aix 账号推送 (DES)'
     ja: 'Ansible user モジュールを使用して Aix アカウントをプッシュする (DES)'
     en: 'Using Ansible module user to push account (DES)'
 
-  Modify params sudo help text:
-    zh: '如果用户存在，可以修改sudo权限'
-    ja: 'ユーザーが存在する場合、sudo権限を変更できます'
-    en: 'If the user exists, sudo permissions can be modified'
-
   Params sudo help text:
     zh: '使用逗号分隔多个命令，如: /bin/whoami,/sbin/ifconfig'
     ja: 'コンマで区切って複数のコマンドを入力してください。例: /bin/whoami,/sbin/ifconfig'
@@ -66,16 +49,6 @@ i18n:
     ja: 'グループを入力してください。複数のグループはコンマで区切ってください（既存のグループを入力してください）'
     en: 'Please enter the group. Multiple groups are separated by commas (please enter the existing group)'
 
-  Params uid help text:
-    zh: '请输入用户ID'
-    ja: 'ユーザーIDを入力してください'
-    en: 'Please enter the user ID'
-
-  Modify sudo label:
-    zh: '修改 sudo 权限'
-    ja: 'sudo 権限を変更'
-    en: 'Modify sudo'
-
   Params home label:
     zh: '家目录'
     ja: 'ホームディレクトリ'
@@ -86,7 +59,3 @@ i18n:
     ja: 'グループ'
     en: 'Groups'
 
-  Params uid label:
-    zh: '用户ID'
-    ja: 'ユーザーID'
-    en: 'User ID'

apps/accounts/automations/push_account/host/posix/main.yml

@@ -9,20 +9,55 @@
         database: passwd
         key: "{{ account.username }}"
       register: user_info
-      failed_when: false
-      changed_when: false
+      ignore_errors: yes  # 忽略错误，如果用户不存在时不会导致playbook失败
 
     - name: "Add {{ account.username }} user"
       ansible.builtin.user:
         name: "{{ account.username }}"
-        uid: "{{ params.uid | int if params.uid | length > 0 else omit }}"
-        shell: "{{ params.shell if params.shell | length > 0 else omit }}"
-        home: "{{ params.home if params.home | length > 0 else '/home/' + account.username }}"
-        groups: "{{ params.groups if params.groups | length > 0 else omit }}"
-        append: "{{ true if params.groups | length > 0 else false }}"
+        shell: "{{ params.shell }}"
+        home: "{{ params.home | default('/home/' + account.username, true) }}"
+        groups: "{{ params.groups }}"
         expires: -1
         state: present
-      when: user_info.msg is defined
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} group"
+      ansible.builtin.group:
+        name: "{{ account.username }}"
+        state: present
+      when: user_info.failed
+
+    - name: "Add {{ account.username }} user to group"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        groups: "{{ params.groups }}"
+      when:
+        - user_info.failed
+        - params.groups
+
+    - name: "Change {{ account.username }} password"
+      ansible.builtin.user:
+        name: "{{ account.username }}"
+        password: "{{ account.secret | password_hash('sha512') }}"
+        update_password: always
+      ignore_errors: true
+      when: account.secret_type == "password"
+
+    - name: remove jumpserver ssh key
+      ansible.builtin.lineinfile:
+        dest: "{{ ssh_params.dest }}"
+        regexp: "{{ ssh_params.regexp }}"
+        state: absent
+      when:
+        - account.secret_type == "ssh_key"
+        - ssh_params.strategy == "set_jms"
+
+    - name: "Change {{ account.username }} SSH key"
+      ansible.builtin.authorized_key:
+        user: "{{ account.username }}"
+        key: "{{ account.secret }}"
+        exclusive: "{{ ssh_params.exclusive }}"
+      when: account.secret_type == "ssh_key"
 
     - name: "Set {{ account.username }} sudo setting"
       ansible.builtin.lineinfile:
@@ -32,61 +67,9 @@
         line: "{{ account.username + ' ALL=(ALL) NOPASSWD: ' + params.sudo }}"
         validate: visudo -cf %s
       when:
-        - user_info.msg is defined or params.modify_sudo
+        - user_info.failed
         - params.sudo
 
-    - name: "Change {{ account.username }} password"
-      ansible.builtin.user:
-        name: "{{ account.username }}"
-        password: "{{ account.secret | password_hash('sha512') }}"
-        update_password: always
-      ignore_errors: true
-      register: change_secret_result
-      when: account.secret_type == "password"
-
-    - name: "Get home directory for {{ account.username }}"
-      ansible.builtin.shell: "getent passwd {{ account.username }} | cut -d: -f6"
-      register: home_dir
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Check if home directory exists for {{ account.username }}"
-      ansible.builtin.stat:
-        path: "{{ home_dir.stdout.strip() }}"
-      register: home_dir_stat
-      when: account.secret_type == "ssh_key"
-      ignore_errors: yes
-
-    - name: "Ensure {{ account.username }} home directory exists"
-      ansible.builtin.file:
-        path: "{{ home_dir.stdout.strip() }}"
-        state: directory
-        owner: "{{ account.username }}"
-        group: "{{ account.username }}"
-        mode: '0750'
-      when:
-        - account.secret_type == "ssh_key"
-        - home_dir_stat.stat.exists == false
-      ignore_errors: yes
-
-    - name: Remove jumpserver ssh key
-      ansible.builtin.lineinfile:
-        dest: "{{ home_dir.stdout.strip() }}/.ssh/authorized_keys"
-        regexp: "{{ ssh_params.regexp }}"
-        state: absent
-      when:
-        - account.secret_type == "ssh_key"
-        - ssh_params.strategy == "set_jms"
-      ignore_errors: yes
-
-    - name: "Change {{ account.username }} SSH key"
-      ansible.builtin.authorized_key:
-        user: "{{ account.username }}"
-        key: "{{ account.secret }}"
-        exclusive: "{{ ssh_params.exclusive }}"
-      register: change_secret_result
-      when: account.secret_type == "ssh_key"
-
     - name: Refresh connection
       ansible.builtin.meta: reset_connection
 
@@ -96,16 +79,14 @@
         login_password: "{{ account.secret }}"
         login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.port }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         become: "{{ account.become.ansible_become | default(False) }}"
         become_method: su
         become_user: "{{ account.become.ansible_user | default('') }}"
         become_password: "{{ account.become.ansible_password | default('') }}"
         become_private_key_path: "{{ account.become.ansible_ssh_private_key_file | default(None) }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "password"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "password"
       delegate_to: localhost
 
     - name: "Verify {{ account.username }} SSH KEY (paramiko)"
@@ -114,10 +95,8 @@
         login_port: "{{ jms_asset.port }}"
         login_user: "{{ account.username }}"
         login_private_key_path: "{{ account.private_key_path  }}"
-        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default(None) }}"
+        gateway_args: "{{ jms_asset.ansible_ssh_common_args | default('') }}"
         old_ssh_version: "{{ jms_asset.old_ssh_version | default(False) }}"
-      when:
-       - account.secret_type == "ssh_key"
-       - check_conn_after_change or change_secret_result.failed | default(false)
+      when: account.secret_type == "ssh_key"
       delegate_to: localhost
 

apps/accounts/automations/push_account/host/posix/manifest.yml

@@ -6,12 +6,6 @@ type:
   - linux
 method: push_account
 params:
-  - name: modify_sudo
-    type: bool
-    label: "{{ 'Modify sudo label' | trans }}"
-    default: False
-    help_text: "{{ 'Modify params sudo help text' | trans }}"
-
   - name: sudo
     type: str
     label: 'Sudo'
@@ -36,23 +30,12 @@ params:
     default: ''
     help_text: "{{ 'Params groups help text' | trans }}"
 
-  - name: uid
-    type: str
-    label: "{{ 'Params uid label' | trans }}"
-    default: ''
-    help_text: "{{ 'Params uid help text' | trans }}"
-
 i18n:
   Posix account push:
     zh: '使用 Ansible 模块 user 执行账号推送 (sha512)'
     ja: 'Ansible user モジュールを使用してアカウントをプッシュする (sha512)'
     en: 'Using Ansible module user to push account (sha512)'
 
-  Modify params sudo help text:
-    zh: '如果用户存在，可以修改sudo权限'
-    ja: 'ユーザーが存在する場合、sudo権限を変更できます'
-    en: 'If the user exists, sudo permissions can be modified'
-
   Params sudo help text:
     zh: '使用逗号分隔多个命令，如: /bin/whoami,/sbin/ifconfig'
     ja: 'コンマで区切って複数のコマンドを入力してください。例: /bin/whoami,/sbin/ifconfig'
@@ -68,16 +51,6 @@ i18n:
     ja: 'グループを入力してください。複数のグループはコンマで区切ってください（既存のグループを入力してください）'
     en: 'Please enter the group. Multiple groups are separated by commas (please enter the existing group)'
 
-  Params uid help text:
-    zh: '请输入用户ID'
-    ja: 'ユーザーIDを入力してください'
-    en: 'Please enter the user ID'
-
-  Modify sudo label:
-    zh: '修改 sudo 权限'
-    ja: 'sudo 権限を変更'
-    en: 'Modify sudo'
-
   Params home label:
     zh: '家目录'
     ja: 'ホームディレクトリ'
@@ -86,9 +59,4 @@ i18n:
   Params groups label:
     zh: '用户组'
     ja: 'グループ'
-    en: 'Groups'
-
-  Params uid label:
-    zh: '用户ID'
-    ja: 'ユーザーID'
-    en: 'User ID'
+    en: 'Groups'

apps/accounts/automations/push_account/host/windows/main.yml

@@ -4,6 +4,10 @@
     - name: Test privileged account
       ansible.windows.win_ping:
 
+#    - name: Print variables
+#      debug:
+#        msg: "Username: {{ account.username }}, Password: {{ account.secret }}"
+
     - name: Push user password
       ansible.windows.win_user:
         fullname: "{{ account.username}}"
@@ -24,4 +28,4 @@
       vars:
         ansible_user: "{{ account.username }}"
         ansible_password: "{{ account.secret }}"
-      when: account.secret_type == "password" and check_conn_after_change
+      when: account.secret_type == "password"

apps/accounts/automations/push_account/host/windows_ad/main.yml

@@ -1,27 +0,0 @@
-- hosts: demo
-  gather_facts: no
-  tasks:
-    - name: Test privileged account
-      ansible.windows.win_ping:
-
-    - name: Push user password
-      community.windows.win_domain_user:
-        name: "{{ account.username }}"
-        password: "{{ account.secret }}"
-        update_password: always
-        password_never_expires: yes
-        state: present
-        groups: "{{ params.groups }}"
-        groups_action: add
-      ignore_errors: true
-      when: account.secret_type == "password"
-
-    - name: Refresh connection
-      ansible.builtin.meta: reset_connection
-
-    - name: Verify password
-      ansible.windows.win_ping:
-      vars:
-        ansible_user: "{{ account.full_username }}"
-        ansible_password: "{{ account.secret }}"
-      when: account.secret_type == "password" and check_conn_after_change

apps/accounts/automations/push_account/host/windows_ad/manifest.yml

@@ -1,25 +0,0 @@
-id: push_account_ad_windows
-name: "{{ 'Windows account push' | trans }}"
-version: 1
-method: push_account
-category:
-  - ds
-type:
-  - windows_ad
-params:
-  - name: groups
-    type: str
-    label: '用户组'
-    default: 'Users,Remote Desktop Users'
-    help_text: "{{ 'Params groups help text' | trans }}"
-
-i18n:
-  Windows account push:
-    zh: '使用 Ansible 模块 win_domain_user 执行 Windows 账号推送'
-    ja: 'Ansible win_domain_user モジュールを使用して Windows アカウントをプッシュする'
-    en: 'Using Ansible module win_domain_user to push account'
-
-  Params groups help text:
-    zh: '请输入用户组，多个用户组使用逗号分隔（需填写已存在的用户组）'
-    ja: 'グループを入力してください。複数のグループはコンマで区切ってください（既存のグループを入力してください）'
-    en: 'Please enter the group. Multiple groups are separated by commas (please enter the existing group)'

apps/accounts/automations/push_account/host/windows_rdp_verify/main.yml

@@ -4,6 +4,10 @@
     - name: Test privileged account
       ansible.windows.win_ping:
 
+#    - name: Print variables
+#      debug:
+#        msg: "Username: {{ account.username }}, Password: {{ account.secret }}"
+
     - name: Push user password
       ansible.windows.win_user:
         fullname: "{{ account.username}}"
@@ -21,11 +25,11 @@
 
     - name: Verify password (pyfreerdp)
       rdp_ping:
-        login_host: "{{ jms_asset.origin_address }}"
+        login_host: "{{ jms_asset.address }}"
         login_port: "{{ jms_asset.protocols | selectattr('name', 'equalto', 'rdp') | map(attribute='port') | first }}"
         login_user: "{{ account.username }}"
         login_password: "{{ account.secret }}"
         login_secret_type: "{{ account.secret_type }}"
-        gateway_args: "{{ jms_gateway | default({}) }}"
-      when: account.secret_type == "password" and check_conn_after_change
+        login_private_key_path: "{{ account.private_key_path }}"
+      when: account.secret_type == "password"
       delegate_to: localhost

apps/accounts/automations/push_account/manager.py

@@ -1,66 +1,74 @@
-from django.utils.translation import gettext_lazy as _
-
 from accounts.const import AutomationTypes
 from common.utils import get_logger
-from common.utils.timezone import local_now_filename
-from ..base.manager import BaseChangeSecretPushManager
-from ...models import PushSecretRecord
+from ..base.manager import AccountBasePlaybookManager
+from ..change_secret.manager import ChangeSecretManager
 
 logger = get_logger(__name__)
 
 
-class PushAccountManager(BaseChangeSecretPushManager):
-
-    @staticmethod
-    def require_update_version(account, recorder):
-        account.skip_history_when_saving = True
-        return False
+class PushAccountManager(ChangeSecretManager, AccountBasePlaybookManager):
 
     @classmethod
     def method_type(cls):
         return AutomationTypes.push_account
 
-    def get_secret(self, account):
-        secret = account.secret
-        if not secret:
-            secret = super().get_secret(account)
-        return secret
+    # @classmethod
+    # def trigger_by_asset_create(cls, asset):
+    #     automations = PushAccountAutomation.objects.filter(
+    #         triggers__contains=TriggerChoice.on_asset_create
+    #     )
+    #     account_automation_map = {auto.username: auto for auto in automations}
+    #
+    #     util = AssetPermissionUtil()
+    #     permissions = util.get_permissions_for_assets([asset], with_node=True)
+    #     account_permission_map = defaultdict(list)
+    #     for permission in permissions:
+    #         for account in permission.accounts:
+    #             account_permission_map[account].append(permission)
+    #
+    #     username_automation_map = {}
+    #     for username, automation in account_automation_map.items():
+    #         if username != '@USER':
+    #             username_automation_map[username] = automation
+    #             continue
+    #
+    #         asset_permissions = account_permission_map.get(username)
+    #         if not asset_permissions:
+    #             continue
+    #         asset_permissions = util.get_permissions([p.id for p in asset_permissions])
+    #         usernames = asset_permissions.values_list('users__username', flat=True).distinct()
+    #         for _username in usernames:
+    #             username_automation_map[_username] = automation
+    #
+    #     asset_usernames_exists = asset.accounts.values_list('username', flat=True)
+    #     accounts_to_create = []
+    #     accounts_to_push = []
+    #     for username, automation in username_automation_map.items():
+    #         if username in asset_usernames_exists:
+    #             continue
+    #
+    #         if automation.secret_strategy != SecretStrategy.custom:
+    #             secret_generator = SecretGenerator(
+    #                 automation.secret_strategy, automation.secret_type,
+    #                 automation.password_rules
+    #             )
+    #             secret = secret_generator.get_secret()
+    #         else:
+    #             secret = automation.secret
+    #
+    #         account = Account(
+    #             username=username, secret=secret,
+    #             asset=asset, secret_type=automation.secret_type,
+    #             comment='Create by account creation {}'.format(automation.name),
+    #         )
+    #         accounts_to_create.append(account)
+    #         if automation.action == 'create_and_push':
+    #             accounts_to_push.append(account)
+    #         else:
+    #             accounts_to_create.append(account)
+    #
+    #         logger.debug(f'Create account {account} for asset {asset}')
 
-    def gen_account_inventory(self, account, asset, h, path_dir):
-        secret = self.get_secret(account)
-        secret_type = account.secret_type
-        if not secret:
-            raise ValueError(_('Secret cannot be empty'))
-        self.get_or_create_record(asset, account, h['name'])
-        new_secret, private_key_path = self.handle_ssh_secret(secret_type, secret, path_dir)
-        h = self.gen_inventory(h, account, new_secret, private_key_path, asset)
-        return h
-
-    def get_or_create_record(self, asset, account, name):
-        asset_account_id = f'{asset.id}-{account.id}'
-
-        if asset_account_id in self.record_map:
-            record_id = self.record_map[asset_account_id]
-            recorder = PushSecretRecord.objects.filter(id=record_id).first()
-        else:
-            recorder = self.create_record(asset, account)
-
-        self.name_recorder_mapper[name] = recorder
-        return recorder
-
-    def create_record(self, asset, account):
-        recorder = PushSecretRecord(
-            asset=asset, account=account, execution=self.execution,
-            comment=f'{account.username}@{asset.address}'
-        )
-        return recorder
-
-    def print_summary(self):
-        print('\n\n' + '-' * 80)
-        plan_execution_end = _('Plan execution end')
-        print('{} {}\n'.format(plan_execution_end, local_now_filename()))
-        time_cost = _('Duration')
-        print('{}: {}s'.format(time_cost, self.duration))
-
-    def get_report_template(self):
-        return "accounts/push_account_report.html"
+    # @classmethod
+    # def trigger_by_permission_accounts_change(cls):
+    #     pass