python / cpython

The Python programming language
https://www.python.org
Other
61.86k stars 29.75k forks source link

Duplicate '.bmp' key in mimetypes.py, maps to both 'image/bmp' and 'image/x-ms-bmp' #88377

Open 3d6f9085-2ec4-4475-85f9-d7f9841df54e opened 3 years ago

3d6f9085-2ec4-4475-85f9-d7f9841df54e commented 3 years ago
BPO 44211
Nosy @terryjreedy, @bitdancer, @serhiy-storchaka, @zooba, @andreasjansson
PRs
  • python/cpython#26300
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields: ```python assignee = None closed_at = None created_at = labels = ['type-bug', 'library', '3.9', '3.10', '3.11'] title = "Duplicate '.bmp' key in mimetypes.py, maps to both 'image/bmp' and 'image/x-ms-bmp'" updated_at = user = 'https://github.com/andreasjansson' ``` bugs.python.org fields: ```python activity = actor = 'serhiy.storchaka' assignee = 'none' closed = False closed_date = None closer = None components = ['Library (Lib)'] creation = creator = 'andreasjansson' dependencies = [] files = [] hgrepos = [] issue_num = 44211 keywords = ['patch'] message_count = 4.0 messages = ['394157', '394679', '401630', '402038'] nosy_count = 5.0 nosy_names = ['terry.reedy', 'r.david.murray', 'serhiy.storchaka', 'steve.dower', 'andreasjansson'] pr_nums = ['26300'] priority = 'normal' resolution = None stage = 'patch review' status = 'open' superseder = None type = 'behavior' url = 'https://bugs.python.org/issue44211' versions = ['Python 3.9', 'Python 3.10', 'Python 3.11'] ```

    3d6f9085-2ec4-4475-85f9-d7f9841df54e commented 3 years ago

    The mime type of .bmp was changed from image/x-ms-bmp in https://github.com/python/cpython/pull/4756.

    https://github.com/python/cpython/pull/3062 then re-introduced image/x-ms-bmp while keeping the newer image/bmp mapping.

    terryjreedy commented 3 years ago

    David and Steve merged the seemingly conflicting PRs.

    serhiy-storchaka commented 2 years ago

    Since this change has a user-visible effect a NEWS entry is required. Just copy it from bpo-22589.

    serhiy-storchaka commented 2 years ago

    It is not clear whether we want to backport this change.

    furkanonder commented 1 year ago

    @serhiy-storchaka Issue seems to resolved. We can close the issue.