Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix handling of user set on the task level #193

Merged
merged 4 commits into from
Apr 22, 2024
Merged

Fix handling of user set on the task level #193

merged 4 commits into from
Apr 22, 2024

Commits on Apr 22, 2024

  1. fix the comment

    @umputun
    umputun committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    24512a2 View commit details
    Browse the repository at this point in the history

  2. Fix incorrect handling of task-level user #192 

    The change eliminates setting the default task.User on the config level,
and moves this responsibility to the runner. Now the runner explicitly
checks the presence of task.User and passes it in.

In addition, runTaskOnHost sets the passed user into activeTask
(a copy of the task used later on). This was done to be able to know what
the actual user is. Currently, the only use is in setting SPOT_REMOTE_USER,
however, it can also be used in any place that needs to know the active remote user.
    @umputun
    umputun committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    ef317d2 View commit details
    Browse the repository at this point in the history

  3. update man and site

    @umputun
    umputun committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    e08ad86 View commit details
    Browse the repository at this point in the history

  4. add a test for user name set on task level

    @umputun
    umputun committed Apr 22, 2024
    Configuration menu
    Copy the full SHA
    ac11d63 View commit details
    Browse the repository at this point in the history