[PR #1066] [MERGED] Fix: panic: reflect: slice index out of range #1752

Closed
opened 2026-03-01 21:52:31 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/1066
Author: @ChristopherHX
Created: 3/21/2022
Status: Merged
Merged: 3/22/2022
Merged by: @mergify[bot]

Base: masterHead: christopherhx/fix/arrayderef


📝 Commits (5)

  • 073dbd6 Fix: panic: reflect: slice index out of range
  • 86cc549 Update interpreter.go
  • 8ab2bdf [no ci] Return null for negative indexes
  • b8e03a3 Add tests for index access
  • 73f2251 Merge branch 'master' into christopherhx/fix/arrayderef

📊 Changes

2 files changed (+7 additions, -0 deletions)

View changed files

📝 pkg/exprparser/interpreter.go (+3 -0)
📝 pkg/exprparser/interpreter_test.go (+4 -0)

📄 Description

panics are very bad


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/nektos/act/pull/1066 **Author:** [@ChristopherHX](https://github.com/ChristopherHX) **Created:** 3/21/2022 **Status:** ✅ Merged **Merged:** 3/22/2022 **Merged by:** [@mergify[bot]](https://github.com/apps/mergify) **Base:** `master` ← **Head:** `christopherhx/fix/arrayderef` --- ### 📝 Commits (5) - [`073dbd6`](https://github.com/nektos/act/commit/073dbd6dc99f891dc67c99b1d7df64c661efb61d) Fix: panic: reflect: slice index out of range - [`86cc549`](https://github.com/nektos/act/commit/86cc549335ca13f2beab23c356d721ed98e04b3f) Update interpreter.go - [`8ab2bdf`](https://github.com/nektos/act/commit/8ab2bdf3de7ec3816bdd5c87935dacf2c1f6cae2) [no ci] Return null for negative indexes - [`b8e03a3`](https://github.com/nektos/act/commit/b8e03a371f0cb6ff86a40cf8e95be044d189fdf2) Add tests for index access - [`73f2251`](https://github.com/nektos/act/commit/73f2251c0ed61d15c21ba97cff091090a4fbb3ab) Merge branch 'master' into christopherhx/fix/arrayderef ### 📊 Changes **2 files changed** (+7 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `pkg/exprparser/interpreter.go` (+3 -0) 📝 `pkg/exprparser/interpreter_test.go` (+4 -0) </details> ### 📄 Description panics are very bad --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 21:52:31 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/act#1752
No description provided.