Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: use multierror #13

Merged
merged 1 commit into from
Feb 9, 2022
Merged

refactor: use multierror #13

merged 1 commit into from
Feb 9, 2022

Conversation

knqyf263
Copy link
Owner

@knqyf263 knqyf263 commented Feb 9, 2022

$ go run cmd/rpmdb/main.go
2022/02/09 12:11:02 3 errors occurred:
        * open ./rpmdb.sqlite: no such file or directory
        * open ./Packages.db: no such file or directory
        * open ./Packages: no such file or directory

exit status 1

@knqyf263 knqyf263 merged commit 9cd0802 into master Feb 9, 2022
@knqyf263 knqyf263 deleted the refactor branch February 9, 2022 10:16
meghfossa added a commit to meghfossa/go-rpmdb that referenced this pull request Apr 11, 2022
* Remove Berkeley DB C-bindings (anchore#3)

* add bdb go util & remove bdb bindings

* add sourcerpm, size, license, and vendor fields

* migrate from wagoodman/go-rpmdb to anchore/go-rpmdb

* keep original pkg/module name

* fix test assertion message

Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>

* verify we are only looking at overflow pages

Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>

* validate hash metadata pages on parse

Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>

* use xerrors and add overflow pagetype constant

Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>

* chore(README): add missing return

* chore: add GitHub Actions (anchore#5)

* Add MODULARITYLABEL Support (anchore#4)

* Add MODULARITYLABEL

* refactoriing deleteIndex

* refactoring entry.go

* [WIP] add verify info

* refactor review

* add data length implimentation

* support array string

* refactor(entry): add a reference

* refactor(entry): replace 4.11 with 4.14

* refactor(entry): update error messages

* refactor: move a validation into hdrblobVerifyRegion

* refactor: add regionTag

* add rdlen check

* add strtaglen verify

* refactor: remove unnecessary cast

* refactor: use camel case

* refactor: update errors

* fix(lint) change  rdlen uint32 to int32

* Fix fuzzer issues

* Add il index check test

* run go mod tidy

Co-authored-by: knqyf263 <knqyf263@gmail.com>

* feat: support RHEL/CentOS 5 (knqyf263#6)

* test(rpmdb): add a failure case

* feat: support RHEL/CentOS 5

* feat: support installed files (knqyf263#7)

* fix(package): skip empty installed files (knqyf263#8)

* Add support for handling RPM databases in NDB format (knqyf263#9)

* Add support for handling RPM databases in NDB format

RPM databases can be either in BerkeleyDB format or optionally in the
newer "native" NDB format. there are linux distributions that switched
their implementation of the rpm database away from BerkeleyDB out of
concerns about future level of maintenance after its license change.

See https://rpm.org/wiki/Releases/4.14.0
See also https://fedoraproject.org/wiki/Changes/NewRpmDBFormat

This adds a simple interface that allows to drive either berkelydb
or NDB format reading.

* Begin error messages with lowercase

Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>

* More protection against potentially malformatted databases

Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>

* Drop use of go-restruct/restruct (knqyf263#10)

* feat: add support for handling RPM database in SQLite3 format (knqyf263#12)

Co-authored-by: knqyf263 <knqyf263@gmail.com>

* refactor: use multierror (knqyf263#13)

* chore: bump up Go to 1.17 (knqyf263#14)

* test: add CBL-Mariner2.0 (knqyf263#15)

Co-authored-by: Alex Goodman <wagoodman@users.noreply.github.com>
Co-authored-by: Teppei Fukuda <knqyf263@gmail.com>
Co-authored-by: Masahiro331 <mur4m4s4.331@gmail.com>
Co-authored-by: Dirk Mueller <dmueller@suse.com>
Co-authored-by: MaineK00n <mainek00n.1229@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant