Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
OpenStackweb
/
summit-api
Public
Notifications
You must be signed in to change notification settings
Fork
2
Star
3
Code
Issues
3
Pull requests
14
Actions
Projects
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Projects
Security and quality
Insights
feat: async file post-processing pipeline for company logos
- #561
#561
Merged
smarcet
merged 13 commits into
main
OpenStackweb/summit-api:main
from
feature/company-logo-postprocessing
OpenStackweb/summit-api:feature/company-logo-postprocessing
Copy head branch name to clipboard
Jun 22, 2026
Conversation
Commits
13
(13)
Checks
Files changed
Merged
feat: async file post-processing pipeline for company logos
#561
smarcet
merged 13 commits into
main
OpenStackweb/summit-api:main
from
feature/company-logo-postprocessing
OpenStackweb/summit-api:feature/company-logo-postprocessing
Copy head branch name to clipboard
Commits
Commits on Jun 19, 2026
feat: async file post-processing pipeline for company logos
Show description for 476846c
smarcet
committed
476846c
View commit details
Copy full SHA for 476846c
Browse repository at this point
docs(swagger): add FileDTO schema and logo/big_logo to company request schemas
Show description for f0f076f
smarcet
committed
f0f076f
View commit details
Copy full SHA for f0f076f
Browse repository at this point
refactor(swagger): move FileDTO schema to its own file
smarcet
committed
7db85e5
View commit details
Copy full SHA for 7db85e5
Browse repository at this point
fix: address CodeRabbit review findings
Show description for 0ea5782
smarcet
committed
0ea5782
View commit details
Copy full SHA for 0ea5782
Browse repository at this point
fix: adversarial review findings
Show description for 4e70016
smarcet
committed
4e70016
View commit details
Copy full SHA for 4e70016
Browse repository at this point
Commits on Jun 22, 2026
refactor: extract duplicate logo-processing helpers in CompanyService
Show description for e53568e
smarcet
committed
e53568e
View commit details
Copy full SHA for e53568e
Browse repository at this point
fix: unlink tempnam file when fopen fails to prevent temp file leak
smarcet
committed
79b0ca3
View commit details
Copy full SHA for 79b0ca3
Browse repository at this point
test: add missing coverage for addCompany logo dispatch and partial-commit paths
Show description for 578c854
smarcet
committed
578c854
View commit details
Copy full SHA for 578c854
Browse repository at this point
fix: restore job retry mechanism and fix FilePostProcessorService return type
Show description for ffb71cb
smarcet
committed
ffb71cb
View commit details
Copy full SHA for ffb71cb
Browse repository at this point
fix: swallow cleanup errors after successful logo upload to prevent duplicate File records on job retry
smarcet
committed
ad3192d
View commit details
Copy full SHA for ad3192d
Browse repository at this point
fix: fail FileProcessingJob immediately on ValidationException and EntityNotFoundException to avoid pointless retries
smarcet
committed
ab3b65a
View commit details
Copy full SHA for ab3b65a
Browse repository at this point
refactor: consolidate file_dto validation rules into FileInfoDTO::validationRules() as single source of truth
smarcet
committed
f845dee
View commit details
Copy full SHA for f845dee
Browse repository at this point
fix: store UploadedFile in variable before file_get_contents to prevent GC race in CI
smarcet
committed
b19ac92
View commit details
Copy full SHA for b19ac92
Browse repository at this point
You can’t perform that action at this time.
