Replace debug IO.puts statements with Logger in PlayerLive.Show (#2287)

[immortal71]

Summary

• replace IO.puts debug statements in copi.owasp.org/lib/copi_web/live/player_live/show.ex
• add require Logger
• use Logger.debug/1 for routine vote/continue-vote flow messages
• use Logger.warning/1 for failed vote/continue-vote operations

Why (Reason)

IO.puts is not suitable for production logging in Elixir apps because it bypasses level-based filtering and logger backends.

Changes

• IO.puts("Continue vote added successfully") -> Logger.debug(...)
• IO.puts("Continue voting failed") -> Logger.warning(...)
• IO.puts("player has voted") -> Logger.debug(...)
• IO.puts("player hasn't voted") -> Logger.debug(...)
• IO.puts("voted successfully") -> Logger.debug(...)
• IO.puts("voting failed") -> Logger.warning(...)

All new log messages include relevant identifiers (player_id, game_id, dealt_card_id) for easier diagnostics.

Validation

• mix compile passes in copi.owasp.org

Closes #2287

[Copilot]

Pull request overview

This PR replaces debug IO.puts statements with proper Logger calls in the PlayerLive.Show module, addressing technical debt from development. IO.puts writes directly to stdout without log levels, timestamps, or structured logging support, making it unsuitable for production. The changes convert 6 debug statements to appropriate Logger calls with added context (player_id, game_id, dealt_card_id) for better diagnostics.

Changes:

• Added require Logger to enable Logger macros
• Replaced successful operation messages with Logger.debug/1 for routine vote/continue-vote flows
• Replaced failure messages with Logger.warning/1 for failed vote operations
• Enhanced log messages to include relevant identifiers for easier troubleshooting

[immortal71]

@sydseter is this good to go ?

[sydseter]

@immortal71 Some of your commits are lacking a verified signature.

[immortal71]

@sydseter done

[immortal71]

@sydseter waiting for the review !!