Fix retry_statuses dropped because list.append() returns None

[mark124]

What

EdFiAPI.get_retry_client() builds its retry-status set as:

statuses=self.lightbeam.config['connection']["retry_statuses"].append(401)

list.append() returns None, so statuses=None reaches ExponentialRetry and the configured retry_statuses (429/500/501/503/504) are dropped - a 429 rate-limit is never retried. It also mutates config['connection']['retry_statuses'] in place, appending another 401 on every call.

Fix

Build a fresh list - a copy of the configured statuses plus 401:

statuses=[*self.lightbeam.config['connection']["retry_statuses"], 401]

One line; fixes both the dropped statuses and the in-place mutation. (The edit also adds a trailing newline at end of file - the second line in the diff.)

Verified

get_retry_client() now retries {429, 500, 501, 503, 504, 401}, and the config list is unchanged across repeated calls.

Addresses the retry-client bug noted in the "Related bug (compounding)" section of #92.

[snyk-io-us]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Licenses	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.