Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

BUG: unicode #705

Merged
merged 2 commits into from
Jun 28, 2023
Merged

BUG: unicode #705

merged 2 commits into from
Jun 28, 2023

Conversation

SantiagoPittella
Copy link
Collaborator

Description

Fixes the bug that generated an outage in mainnet due to a contract with unicode characters.

closes #698

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.

@juanbono juanbono enabled auto-merge June 28, 2023 22:08
@juanbono juanbono added this pull request to the merge queue Jun 28, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #705 (4fae723) into main (ec11683) will increase coverage by 0.13%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #705      +/-   ##
==========================================
+ Coverage   91.97%   92.11%   +0.13%     
==========================================
  Files          52       52              
  Lines       11341    11339       -2     
==========================================
+ Hits        10431    10445      +14     
+ Misses        910      894      -16     
Impacted Files Coverage Δ
...re/contract_address/deprecated_contract_address.rs 93.71% <100.00%> (+7.59%) ⬆️

Merged via the queue into main with commit 9c84afb Jun 28, 2023
@juanbono juanbono deleted the support-unicode branch July 1, 2023 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cairo 0 class hash contains the unicode bug
4 participants