Skip to content

Fix for Candidate Not Iterable Error #1082

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

Merged
HarithaVattikuti merged 5 commits into from
Apr 17, 2025
+91 −9
Merged

Fix for Candidate Not Iterable Error #1082

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1795a82
candidates not iterable
aparnajyothi-y Apr 14, 2025
93e3e02
update the error message
aparnajyothi-y Apr 14, 2025
7faf95d
update error to debug
aparnajyothi-y Apr 14, 2025
cc6c47b
update debug to info
aparnajyothi-y Apr 14, 2025
1032141
error message updates
aparnajyothi-y Apr 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
error message updates
  • Loading branch information
@aparnajyothi-y
aparnajyothi-y committed Apr 14, 2025
commit 1032141ebd84945eed09831feceb5c034f5d40c0
17 changes: 8 additions & 9 deletions dist/setup/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97484,10 +97484,9 @@ function getManifest() {
throw new Error('The repository manifest is invalid or does not include any valid tool release (IToolRelease) entries.');
}
catch (err) {
core.debug('Failed to fetch the manifest from the repository API.');
core.debug('Fetching the manifest via the API failed.');
if (err instanceof Error) {
core.debug(`Error message: ${err.message}`);
core.debug(`Error stack: ${err.stack}`);
core.debug(err.message);
}
else {
core.error('An unexpected error occurred while fetching the manifest.');
Expand All @@ -97498,17 +97497,17 @@ function getManifest() {
}
exports.getManifest = getManifest;
function getManifestFromRepo() {
core.info(`Getting manifest from ${MANIFEST_REPO_OWNER}/${MANIFEST_REPO_NAME}@${MANIFEST_REPO_BRANCH}`);
core.debug(`Getting manifest from ${MANIFEST_REPO_OWNER}/${MANIFEST_REPO_NAME}@${MANIFEST_REPO_BRANCH}`);
return tc.getManifestFromRepo(MANIFEST_REPO_OWNER, MANIFEST_REPO_NAME, AUTH, MANIFEST_REPO_BRANCH);
}
exports.getManifestFromRepo = getManifestFromRepo;
function getManifestFromURL() {
return __awaiter(this, void 0, void 0, function* () {
core.info('Falling back to fetching the manifest using raw URL.');
core.debug('Falling back to fetching the manifest using raw URL.');
const http = new httpm.HttpClient('tool-cache');
const response = yield http.getJson(exports.MANIFEST_URL);
if (!response.result) {
throw new Error(`Unable to get manifest from ${exports.MANIFEST_URL}. HTTP status: ${response.statusCode}`);
throw new Error(`Unable to get manifest from ${exports.MANIFEST_URL}`);
}
return response.result;
});
Expand Down Expand Up @@ -97563,16 +97562,16 @@ function installCpythonFromRelease(release) {
if (err instanceof tc.HTTPError) {
// Rate limit?
if (err.httpStatusCode === 403) {
core.error(`Received HTTP status code 403 (Forbidden). This usually indicates that the request is not authorized. Please check your credentials or permissions.`);
core.error(`Received HTTP status code 403. This indicates a permission issue or restricted access.`);
}
else if (err.httpStatusCode === 429) {
core.info(`Received HTTP status code 429 (Too Many Requests). This usually indicates that the rate limit has been exceeded. Please wait and try again later.`);
core.info(`Received HTTP status code 429. This usually indicates the rate limit has been exceeded`);
}
else {
core.info(err.message);
}
if (err.stack) {
core.info(err.stack);
core.debug(err.stack);
}
}
throw err;
Expand Down
19 changes: 8 additions & 11 deletions src/install-python.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,9 @@ export async function getManifest(): Promise<tc.IToolRelease[]> {
'The repository manifest is invalid or does not include any valid tool release (IToolRelease) entries.'
);
} catch (err) {
core.debug('Failed to fetch the manifest from the repository API.');
core.debug('Fetching the manifest via the API failed.');
if (err instanceof Error) {
core.debug(`Error message: ${err.message}`);
core.debug(`Error stack: ${err.stack}`);
core.debug(err.message);
} else {
core.error('An unexpected error occurred while fetching the manifest.');
}
Expand All @@ -74,7 +73,7 @@ export async function getManifest(): Promise<tc.IToolRelease[]> {
}

export function getManifestFromRepo(): Promise<tc.IToolRelease[]> {
core.info(
core.debug(
`Getting manifest from ${MANIFEST_REPO_OWNER}/${MANIFEST_REPO_NAME}@${MANIFEST_REPO_BRANCH}`
);
return tc.getManifestFromRepo(
Expand All @@ -86,14 +85,12 @@ export function getManifestFromRepo(): Promise<tc.IToolRelease[]> {
}

export async function getManifestFromURL(): Promise<tc.IToolRelease[]> {
core.info('Falling back to fetching the manifest using raw URL.');
core.debug('Falling back to fetching the manifest using raw URL.');

const http: httpm.HttpClient = new httpm.HttpClient('tool-cache');
const response = await http.getJson<tc.IToolRelease[]>(MANIFEST_URL);
if (!response.result) {
throw new Error(
`Unable to get manifest from ${MANIFEST_URL}. HTTP status: ${response.statusCode}`
);
throw new Error(`Unable to get manifest from ${MANIFEST_URL}`);
}
return response.result;
}
Expand Down Expand Up @@ -149,17 +146,17 @@ export async function installCpythonFromRelease(release: tc.IToolRelease) {
// Rate limit?
if (err.httpStatusCode === 403) {
core.error(
`Received HTTP status code 403 (Forbidden). This usually indicates that the request is not authorized. Please check your credentials or permissions.`
`Received HTTP status code 403. This indicates a permission issue or restricted access.`
);
} else if (err.httpStatusCode === 429) {
core.info(
`Received HTTP status code 429 (Too Many Requests). This usually indicates that the rate limit has been exceeded. Please wait and try again later.`
`Received HTTP status code 429. This usually indicates the rate limit has been exceeded`
);
} else {
core.info(err.message);
}
if (err.stack) {
core.info(err.stack);
core.debug(err.stack);
}
}
throw err;
Expand Down