This repository has been archived by the owner on Oct 6, 2022. It is now read-only.

Fix nil panic on null environment variable for step #8

Open

mach6 wants to merge 1 commit into drone-runners:master from mach6:mach6_fix_nil_panic

mach6 commented Sep 7, 2022

The following in a .drone.yml will cause the drone-runner-exec process to crash (nil panic) ..

steps:
- name: crash-runner
  environment:
    BURN_IT_DOWN: null

This PR fixes it and a similar issue for engine.Secret

Note: both changes are already present in drone-runner-docker, drone-runner-ssh and drone-runner-kube.


          Fix nil panic on null environment variable for step

5889acd

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet