dosubot[bot] commented on issue #8624:
URL: 
https://github.com/apache/incubator-devlake/issues/8624#issuecomment-3463495916

   <!-- Greeting -->
   Hi @coocoobau! I'm [Dosu](https://go.dosu.dev/dosubot) and I’m helping the 
incubator-devlake team.
   
   <!-- Answer -->
   The panic happens because PR #8583 changed the `PublishedAt` field in the 
`GithubRelease` model to a pointer (`*time.Time`), but the conversion logic in 
`release_convertor.go` dereferences it without a nil check, causing a crash 
when the value is null (as with draft releases) [see PR 
#8583](https://github.com/apache/incubator-devlake/pull/8583). There’s 
currently no safeguard in the conversion function to skip or handle releases 
with a null `published_at` [see 
code](https://github.com/apache/incubator-devlake/blob/2c15ad880dce830c6512fa58039527cbf4745b8b/backend/plugins/github/tasks/release_convertor.go).
   
   To avoid the panic, releases with a null `published_at` (drafts) should be 
skipped during transformation. There’s no documentation or built-in handling 
for this in the current codebase. The recommended approach is to add a check in 
the conversion logic to skip rows where `PublishedAt` is nil.
   
   
   <!-- Dosu Comment Footer -->
   *To reply, just mention [@dosu](https://go.dosu.dev/dosubot).*
   
   ---
   <sup>How did I do? 
[Good](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=hallucination)
 | [Report 
🐛](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/50d5acc4-5a8c-44ef-81fd-ffa13dc5f931?feedback_type=other)</sup>&nbsp;&nbsp;[![Chat
 with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/b4e8e847-d479-4541-83a8-d88d83fea5c9/ask?utm_source=githu
 b)&nbsp;[![Join 
Discord](https://img.shields.io/badge/join-5865F2?logo=discord&logoColor=white&label=)](https://go.dosu.dev/discord-bot)&nbsp;[![Share
 on 
X](https://img.shields.io/badge/X-share-black)](https://twitter.com/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/incubator-devlake/issues/8624)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to