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

vsp: don't set/use a default policy #2129

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

vsp: don't set/use a default policy #2129

wants to merge 2 commits into from

Commits on Feb 28, 2022

  1. vsp: don't set/use a default policy 

    This avoids the possibility of using a previously created vsp client
that has a policy different from the one intended by the caller.

In most cases already, the caller has a policy defined that should be
used rather than some previously set default policy. The only other
case where a policy to be used isn't readily handy is in the json-rpc
handler for the processUnmanagedTicket method. That method uses the
vsp client set in config and should thus use the policy set in config.
    @itswisdomagain
    itswisdomagain committed Feb 28, 2022
    Configuration menu
    Copy the full SHA
    6b3bc7a View commit details
    Browse the repository at this point in the history

  2. set mixedSplitAccount, changeAccount in purchaseTicket json-rppc

    @itswisdomagain
    itswisdomagain committed Feb 28, 2022
    Configuration menu
    Copy the full SHA
    ad36dd9 View commit details
    Browse the repository at this point in the history