imp: check order in validate basic of MsgRegisterInterchainAccount

Description

Also found while testing with hermes: a valid MsgRegisterInterchainAccount messages can be constructed and submitted with NONE order. The channel will not be created because an error is returned in ValidateBasic of MsgChannelOpenInit, but I thought it would be nicer to do the same check in the ValidateBasic of MsgRegisterInterchainAccount.
However, I was also thinking that maybe we actually shouldn't do this check at all in ValidateBasic and just set the order to ORDERED if the value passed in registerInterchainAccount is NONE. That way I think we preserve backwards compatibility better, because upstream users don't need to change the existing code they may have creating a MsgRegisterInterchainAccount, otherwise, they will have to change their code and set a valid ordering.
We could leave the ValidateBasic check of this PR and merge it to main, but not backport; and then in release/v8.1.x we could implement the other solution.
Looking forward to reading people's thoughts.
closes: #XXXX
Commit Message / Changelog Entry
type: commit message

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (59ac9b2) 81.16% compared to head (458e311) 81.17%.
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5671   +/-   ##
=======================================
  Coverage   81.16%   81.17%
=======================================
  Files         199      199
  Lines       15278    15281    +3
=======================================
+ Hits        12401    12404    +3
  Misses       2408     2408
  Partials      469      469

Files
Coverage Δ

...ps/27-interchain-accounts/controller/types/msgs.go
100.00% <100.00%> (ø)

modules/core/04-channel/types/channel.go
77.21% <100.00%> (ø)


Slow production queries

I just had a look at pg_stat_statements in production, after having reset it earlier this morning, which told the following (times in ms):
graph=> select queryid,  calls, max_time, mean_time, rows from pg_stat_statements where userid = 16388 order by total_time desc limit 20;
  queryid   |  calls  | max_time  |      mean_time      |  rows
------------+---------+-----------+---------------------+---------
  980822881 |    3755 |  8002.499 |    1907.71453075899 |    3755
<PHONE_NUMBER> |     429 | 10950.373 |    6131.52871095571 |      42
<PHONE_NUMBER> |     429 |  9740.047 |    5805.78927039627 |     429
<PHONE_NUMBER> |   35984 | 39005.558 |     60.594052023122 |   35984
<PHONE_NUMBER> |  546734 |   100.432 |    3.25583959475709 |  546734
<PHONE_NUMBER> |   85554 |    60.109 |    2.66186966126658 |   85552
<PHONE_NUMBER> |  160113 |     25.98 |   0.948936688463772 |  160113
<PHONE_NUMBER> |  104364 |    48.176 |    1.00678212793686 |  104364
<PHONE_NUMBER> |   37428 |    26.697 |    1.38165434968472 |   37428
  569752855 |  797553 |    25.097 |  0.0519043035384478 |  516542
<PHONE_NUMBER> |   67038 |    24.628 |   0.518791327306904 |  120016
<PHONE_NUMBER> |   34058 |   642.553 |   0.688873803511653 |   99728
<PHONE_NUMBER> |   66940 |    21.029 |   0.269674544368087 |       0
<PHONE_NUMBER> |   23964 |    18.622 |   0.536059714571858 |   23964
<PHONE_NUMBER> |  270421 |     13.75 |  0.0348788074890626 |  270421
<PHONE_NUMBER> |       7 |  2027.682 |<PHONE_NUMBER>28571 |     350
<PHONE_NUMBER> |    3867 |    16.917 |    1.62344737522628 |    3867
<PHONE_NUMBER> |   34529 |     11.43 |   0.171716730863912 |   22127
  512133718 |    1701 |    24.472 |    2.75001234567901 |    3869
<PHONE_NUMBER> | 1190998 |     5.479 | 0.00384790234744306 | 1190998

graph=> select query from pg_stat_statements where queryid = 980822881;
                  query
------------------------------------------
 SELECT attempt_chain_head_update($1, $2)

graph=> select query from pg_stat_statements where queryid =<PHONE_NUMBER>;
                                  query
--------------------------------------------------------------------------
                                                                         +
 WITH                                                                    +
   events_for_block_and_subgraph AS                                      +
     (SELECT distinct h.event_id                                         +
        FROM entity_history h, event_meta_data m                         +
       WHERE h.event_id = m.id                                           +
         AND m.source = $1                                               +
         AND h.subgraph = $2)                                            +
 SELECT                                                                  +
   h.subgraph,                                                           +
   h.entity,                                                             +
   h.entity_id,                                                          +
   (CASE                                                                 +
      WHEN h.op_id = ? THEN ?                                            +
      WHEN h.op_id in (?,?) THEN ?                                       +
    END) as change                                                       +
 FROM entity_history h                                                   +
 WHERE h.event_id in (select event_id from events_for_block_and_subgraph)+
 ORDER BY h.event_id desc

graph=> select query from pg_stat_statements where queryid =<PHONE_NUMBER>;
                query
-------------------------------------
 SELECT revert_block($1, $2, $3, $4)

graph=> select query from pg_stat_statements where queryid =<PHONE_NUMBER>;
                              query
------------------------------------------------------------------
 SELECT COUNT(*) FROM "entities" WHERE "entities"."subgraph" = $1

graph=> select query from pg_stat_statements where queryid =<PHONE_NUMBER>;
                                            query

--------------------------------------------------------------------------------
-------------
 select hash, number from ethereum_blocks where network_name=? order by number d
esc limit ?;

The first on is addressed by PR #786; the next two deal with block reversions, and I am surprised there are that many of them; the one after that (counting entities in a subgraph) uses the subgraph_idx, and is likely caused by the size of that index. The last one does a full table scan on ethereum_blocks, and might benefit from an index.
I'd like to first see PR #786 in prod, and see how the index it introduces affects other queries. We should definitely look into the two queries around block reversion.

Just to gather more data, update from today, after deploying the attempt_chain_head_update; the statistics were cleared out after the update, and cover roughly 24h:
graph=> select queryid,  calls, max_time, mean_time, rows from pg_stat_statements where userid = 16388 order by total_time desc limit 20;
  queryid   |  calls  | max_time  |      mean_time      |  rows
------------+---------+-----------+---------------------+---------
<PHONE_NUMBER> |    5214 | 17934.143 |    10341.0926731876 |    1047
<PHONE_NUMBER> |    5214 | 15865.148 |    9611.49990295356 |    5214
<PHONE_NUMBER> |  141473 | 23121.581 |<PHONE_NUMBER>81731 |  141473
<PHONE_NUMBER> | 1487900 |   203.629 |    4.25120342697742 | 1487900
<PHONE_NUMBER> |  940727 |    60.929 |     1.6846618625808 |  940727
<PHONE_NUMBER> |  408413 |    43.498 |    1.56461543094854 |  408413
<PHONE_NUMBER> |  351153 |    70.385 |    1.65680402274791 |  351153
<PHONE_NUMBER> |  589685 |    41.615 |   0.613163114205039 | 1075805
<PHONE_NUMBER> |      56 |  5394.338 |    4424.48189285714 |      56
<PHONE_NUMBER> |  589428 |    36.349 |   0.322109160067053 |       6
  569752855 | 3097421 |    57.322 |  0.0538456732229801 | 3037581
<PHONE_NUMBER> |   36845 |   232.292 |    3.26119981001493 |   31162
<PHONE_NUMBER> |   41690 |    76.645 |    2.67765891100983 |   41690
<PHONE_NUMBER> | 2337370 |    13.395 |  0.0428984088954677 | 2337370
<PHONE_NUMBER> |  126661 |   384.367 |   0.766809799385767 |  334078
  980822881 |   35261 |    33.026 |    2.35682476390343 |   35261
<PHONE_NUMBER> |   93916 |    34.757 |   0.690595670599247 |   93916
<PHONE_NUMBER> | 7653954 |    19.803 | 0.00439560964176195 | 7653954
<PHONE_NUMBER> |  126191 |    18.691 |    0.13361418009208 |   69930
<PHONE_NUMBER> |   21432 |    48.773 |   0.773234182530798 |   42864

Observations:

attempt_chain_head_update isn't a problem anymore
the biggest problem are with the first two queries to do with block reversion
the only other problem is with counting the entities in a subgraph
it's surprising how many block reversions there were in a 24h period. The number of calls in the first two rows reflects the number of subgraphs * number of eth blocks reverted, but it means there were almost 100 block reversions during that time

The number of block reorgs isn't too surprising. Most of them will be reorgs of 1, maybe 2, blocks. Quote from https://blog.ethereum.org/2015/08/08/chain-reorganisation-depth-expectations/:

With Ethereum having a short target block time of 15s, this actually happens naturally rather often. Because it takes time for the blocks to percolate through the network, it’s easy for different parts of the network to have a different final block (or two, or perhaps even three) in normal operation since the miners often come up with them at roughly the same time. This is what we might call ephemeral forking. Indeed, many of the ommers (né uncles) that you see in stats.ethdev.com were once assumed by some nodes to be the final block in canonical chain.

Another snapshot from today. showing data collected over the last 48 hours or so:
graph=> select queryid,  calls, max_time, mean_time, rows from pg_stat_statements where userid = 16388 order by total_time desc limit 20;
  queryid   |  calls   | max_time  |      mean_time      |   rows
------------+----------+-----------+---------------------+----------
<PHONE_NUMBER> |    10446 |  37765.74 |    18201.3235248898 |     2833
<PHONE_NUMBER> |    10446 | 33313.655 |    16804.4008807199 |    10446
<PHONE_NUMBER> |  4292588 |   669.669 |    13.6020353348611 |  4292588
<PHONE_NUMBER> |   166129 | 29562.348 |    77.8048388481248 |   166129
<PHONE_NUMBER> |  1762788 |   155.642 |    2.80871232331974 |  1762788
  722038960 |  1055512 |    62.746 |    2.58157191201997 |  1055512
<PHONE_NUMBER> |   664297 |    69.186 |    1.75788196243549 |   664297
<PHONE_NUMBER> |      183 | 42812.966 |    6143.56003278688 |      183
  525579649 |  1103155 |     57.83 |   0.866227706895202 |  2014519
<PHONE_NUMBER> |   220297 |    58.312 |    3.22172217960301 |   215461
  569752855 |  7770799 |   255.644 |  0.0714504124994086 |  7195382
<PHONE_NUMBER> |  1097770 |     46.73 |    0.39546127877425 |      132
  980822881 |    58386 |    90.879 |    3.86207354502791 |    58386
  163623117 |   144426 |  1255.257 |    1.28076599088807 |   946252
<PHONE_NUMBER> |  4315787 |    32.099 |   0.039131339892349 |  4315787
<PHONE_NUMBER> |    64542 |    48.146 |    2.49185423445197 |    64542
<PHONE_NUMBER> |    41379 |    38.325 |     2.1108491505353 |   413790
<PHONE_NUMBER> | 14865206 |    36.315 | 0.00420023550295861 | 14865206
<PHONE_NUMBER> |    52199 |    46.174 |    1.14319686200884 |    52199
<PHONE_NUMBER> |    35489 |    42.909 |     1.6779876017921 |   212934

The query<PHONE_NUMBER> is upsert of entities, and is so high in the table because we do a lot of them, not because any individual upsert takes long. The top two queries are block reversion, and there's a clear trend for them to slow down (roughly 3x slower than they were the first time I collected this data)

It feels like this is less of an issue and more of a note. Should we close it in favor of a more actionable issue?

Yeah, it is, but I don't know where else to track this data, since the progression is also useful information. The fix for this issue would be to (being a bit arbitrary), that no query takes more than 1-2s, that any query slower than ~ 100ms has a good reason to be slow, and that the number of times each query is run is as small as it reasonably can (i.e., we understand and agree why we have to run a certain query that often)

Closing this since this particular issue isn't really actionable.


chore: Refactor images patterns

Create gc-complex-img and gc-simple-img pattern
Update posts to adopt new image patterns

@arianna-merritt  The new pattern is supposed to simplify the use of complex images. I was wondering if attribute like file, caption, summary and content are enough self-explanatory compare to src, caption, summary and details.
{% include components/gc-complex-img.html
	 alt="A long description can be found after the image."
	 file=""
	 caption=""
	 summary=""
	 content=""
%}

or
{% include components/gc-complex-img.html
	 alt="A long description can be found after the image."
	 src=""
	 caption=""
	 summary=""
	 details=""
%}


[BugFix] Fix the check the state of opening sink

To improve the performance, the sink will call try_open() in prepare() of OlapTableSinkOperator.
And then call open_wait() in the push_chunk() to wait for the sink opened.
If the chunk is empty, the pull_chunk() has been not called.
So we should call open_wait() in set_finishing() to wait for the sink opened.
Otherwise, the close() and open() will be overlapped and cause an error.
What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

run starrocks_be_unittest

sh starrocks_be_build


patch lax.axis_index, add warning about soft_pmap

This PR makes lax.axis_index no longer raise a NotImplementedError, which we added in #1682.
We don't have great CI tools to test the hard multi-device nested-pmap cases of lax.axis_index, but for posterity here's the kind of test I ran in colab:
import jax
from jax import lax

def f(a):
  return jax.pmap(jax.pmap(lambda x: lax.axis_index('i'), 'j'), 'i')(a)
print(f(onp.ones((2, 2))))

def f(a):
  return jax.pmap(jax.pmap(lambda x: lax.axis_index('j'), 'j'), 'i')(a)
print(f(onp.ones((2, 2))))

def f(a):
  return jax.pmap(jax.pmap(jax.pmap(lambda x: lax.axis_index('i'), 'k'), 'j'), 'i')(a)
print(f(onp.ones((2, 2, 2))))

def f(a):
  return jax.pmap(jax.pmap(jax.pmap(lambda x: lax.axis_index('j'), 'k'), 'j'), 'i')(a)
print(f(onp.ones((2, 2, 2))))

def f(a):
  return jax.pmap(jax.pmap(jax.pmap(lambda x: lax.axis_index('k'), 'k'), 'j'), 'i')(a)
print(f(onp.ones((2, 2, 2))))

The only test failure was, I think, a fluke.


OmniSharp server is not running.

when i try .NET: generate assets for Build and Debug. Its saying Unable to generate assets to build and debug. OmniSharp server is not running. anybody dont know why?

Could you share the OmniSharp log from the Output pane?

PLEASE SHOW HOW TO FIX OMNI SHARP NOT WORKING

@LangerFall  If you press ctrl+shift+p to open the VS Code command palette and type "Select Project", do you see your project listed in the dropdown? if you are able to select it, does it open as expected?
For instance:

After choosing "OmniSharp: Select Project" I am shown:

Reinstall the dot net runtime and sdk worked

Closing due to lack of activity.


"Back to search results" link on mobile is too small

Description

"Back to search results" link should be at least 40px on mobile to be tappable.
Reproduction

Search for something on search-staging.openverse.engineering using mobile screen.
Open one of the results.
You should see "Back to search results" link that only has 28px-high tappable area.

Screenshots

Resolution

[ ] 🙋 I would be interested in resolving this bug.

Hey there! I see from your readme that you don't require permission ahead of time to work on issues, but I just wanted to let you know that I can tackle this one.


Sprint 2

Checklist

Assignment
Points
Grade
Evidence

JSON Object & Arrays

Input and Output

Data Types

Square and Red Squares

Game Work

Total

Skill
Points
Grade
Evidence

Work Habits (Analytics)

Advocacy

Beyond Perfunctory Memory

Total

JSON Objects

Input

Output

Data Types

Any Game Progress you have done

Home Page
https://ethanwong2008.github.io/ethan_2027/


Extension not passing Credentials by using local html page

Hello there ,
i have created html page , which present multiple pages using iframe. the extension doesnt pass now the http basic authenication anymore after chrome update.
but if i go directly to any link of those , the credintials works perfectly.
Is there any solution?
thanks

Hi There,
Sadly no, there's nothing that I can do, I am 100% dependent on the browser to ask for the credentials.
The browser mechanism isn't triggered in case of iframes or any kind of embedded content (see https://github.com/krtek4/MultiPass/issues/65 for the same issues with images for exemple). The reason is security.
Best,


Clear presentation container on demand (client will send request to check if current sequence on server has changed)

Implementation in c418d32


SAKIII-5229 Don't recursively embed pages

https://jira.sakaiproject.org/browse/SAKIII-5229

I pushed an alternative approache where de pageviewer preview is being disabled, closing this one.
https://github.com/sakaiproject/3akai-ux/pull/1835


Profile Pictures

We need to implement Profile Pictures feature so our users will be able to upload their custom images instead of displaying the simple Avatar.
If user didn't add any image, a default one will appear instead.
Useful links:

https://www.npmjs.com/package/react-images-uploading
https://firebase.google.com/docs/storage/web/start

So at the end it should look like this


[HOLD for payment 2024-02-07] [$500] Chat - When user goes offline, the download button do not disappear for other file types

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.32-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Pre-requisite: the user must be logged in and have sent an attachment that is not an image or PDF, such as .mp4 or .doc files

Go to the chat where you have sent the attachment (.mp4, .doc, etc.)
Go offline

Expected Result:
The download button for other type of files should disappear, as it does for images and PDF
Actual Result:
The download button does not disappear when the user goes offline
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/942a8722-dd74-4bfb-99b4-d3cc0f56b8c1
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c2f068b75b4b0950
Upwork Job ID:<PHONE_NUMBER>532508672
Last Price Increase: 2024-01-25
Automatic offers:

ishpaul777 | Reviewer | 28123937
Krishna2323 | Contributor | 28123938

Paid out, closing.


There's no possible way to include tracking categories on LineItem elements

Per the docs at http://developer.xero.com/documentation/api/invoices/, a <LineItem> can contain a <Tracking> element which is a collection of <TrackingCategory> elements, like this:
<LineItem>
  <Description>Consulting services as agreed</Description>
  <Quantity>5.0000</Quantity>
  <UnitAmount>99</UnitAmount>
  <AccountCode>200</AccountCode>
  <Tracking>
      <TrackingCategory>
        <Name>Activity/Workstream</Name>
        <Option>Onsite consultancy</Option>
      </TrackingCategory>
  </Tracking>
</LineItem>

As far as I can see, there's no way to create that <LineItem> as a Daursu/xero model at present. If we pass in an array or collection of TrackingCategory models as a property of the LineItem, the model property (and the name of element representing the collection in the XML representation of the model) automatically gets called TrackingCategories, which is wrong - it should be Tracking. We need a way to override the tag names Pluralizer automatically comes up with for elements representing collections, and insist that no, Tracking is the plural of TrackingCategory, not TrackingCategories.

I will look into it.

I've created a new model Daursu\Xero\Models\Tracking. I've set the following properties:
/**
	 * The name of the entity
	 *
	 * @var string
	 */
	protected static $entity = 'Tracking';

	/**
	 * The singular name of the entity
	 *
	 * @var string
	 */
	protected static $entity_singular = 'TrackingCategory';

This should take care of serialization to XML.
Now with the following code:
$lineItem = new LineItem([
			'Description' => 'Consulting services as agreed',
			'Quantity'    => 5.0,
			'UnitAmount'  => 99,
			'AccountCode' => 200,
		]);

		$tracking = Tracking::newCollection();
		$lineItem->Tracking = $tracking;

		$tracking->push([
			'Name'   => 'Activity/Workstream',
			'Option' => 'Onsite consultancy',
		]);

		$tracking->push([
			'Name'   => 'Remote Support',
			'Option' => 'Remote consultancy',
		]);

		print_r($lineItem->toXML());

I get the following XML output:

<?xml version="1.0" encoding="UTF-8"?>
<LineItems>
   <LineItem>
      <Description>Consulting services as agreed</Description>
      <Quantity>5</Quantity>
      <UnitAmount>99</UnitAmount>
      <AccountCode>200</AccountCode>
      <Tracking>
         <TrackingCategory>
            <Name>Activity/Workstream</Name>
            <Option>Onsite consultancy</Option>
         </TrackingCategory>
         <TrackingCategory>
            <Name>Remote Support</Name>
            <Option>Remote consultancy</Option>
         </TrackingCategory>
      </Tracking>
   </LineItem>
</LineItems>

I hope this helps.

Cool, that should work.


"Cannot find type" build errors when raw_value, alloc features enabled, std disabled

There are many "Cannot find type" errors when building with raw_value, alloc features enabled, std disabled. Mostly concerning Box, ToOwned and String types, traits.
Ok with version 1.0.73.
Not ok with 1.0.74, 1.0.75.

Sorry about the breakage. This is fixed in 1.0.76.


Fix #9542 (allow files under node_modules to be included in the compilation).

Fixes #9542 . It took me about an hour to fix the issue, and another 4 to figure out why the baselines were dropping files for the test changes 😩 (see comment on changes in projectsRunner.ts).
I'm still verifying the changes to a couple of existing baselines are correct, but wanted to get this out for review now.

OK. The other baseline changes are as expected due to the rather non-obvious behavior described in the comment at https://github.com/Microsoft/TypeScript/blob/master/src/harness/compilerRunner.ts#L95 . So this should be good to go.

👍


Fix contextual discrimination for omitted members

In my original PR #43633, choices for optional properties that could be discriminatory were chosen from the union type, which in turn meant that the possible omitted properties to check for discrimination was taken from the intersection of properties across all types. This is the incorrect way to check, when it's actually the union of all properties that should be checked for discrimination. See the additions to the tests for details.
Two stylistic notes about this PR. In principle the code should be going from intersection to union, but it proved more difficult to do given the current functions I found available. As a result:

I couldn't find the best way to get the unique strings of an array like [...new Set()] would, so I created a small helper function, but didn't put it in a great place. Given the lack of documentation in checker, I'm not sure where to put it, or if there are better utilities I can use to achieve the same effect.
This function was relatively hard to parse before. Arguably the first PR didn't help, but it kept it at roughly the same complexity. This PR makes it much worse. Ideally I'd like to strip out some common functionality, or decompose the nested loops, but from looking through checker that seems less the preferred style. Basically, this works, but I don't love it, and feedback is welcome.

Given your initial suggestions and the review of the past PR, @weswigham seems poised to review this the quickest.
Fixes #41759

I'm not sure how to interact with typescript bot, but the previous PR seems to have introduced a stack overflow, which should be addressed at least in tandem with this fix.

Hey @weswigham , the one user who reported a stack overflow hasn't followed up to debug, so I'm hoping it's a non issue. I don't love the style of this code, primarily because it's not clear how best to utilize sets in checker.ts, or when it's reasonable to factor out functions, or where to put utilities like uniqueStrings. Any general comments you have on this, or the meat of the commit would be much appreciated.

This PR seems to have regressed our check times on material UI drastically.


security problem

express-pouchdb server security issue:
anyone with "pouchdb-authentication"  module can create admin user without any prior authentication .eg.
let db = new PouchDB(''http://localhost:3000/testdb');
db.signUpAdmin('batman', 'brucewayne', function (err, response) {
  // etc.

});
with that, everybody can get control over express-pouchdb server.
do you think is it ok??

solved it by modifiying  'express-pouchdb/lib/routes/authorization.js' file by adding following lines after line 13.
 app.get('/_membership',  requiresServerAdmin);
 app.put('/_node/node1@<IP_ADDRESS>/_config/admins/:key',  requiresServerAdmin);
 app.delete('/_node/node1@<IP_ADDRESS>/_config/admins/:key',  requiresServerAdmin);

don't konw is it right or wrong but i saved my day....

#407

Is pouchdb-server basically unusable with this bug still unaddressed?


Fixing squid: S00122 Statement should be on separate lines 200 min

This pull request is focused on resolving occurrences of Sonar rule
squid:S00122 - “ Staement should be on separate lines”.
This PR will reduce 200 min TD:
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:S00122
Please let me know if you have any questions.
Fevzi Ozgul

Coverage decreased (-1.1%) to 38.846% when pulling 63f7bd9002981a998d0374c06f0f8262b05da7c2 on DevFactory:release/Statements_should_be_on_separate_lines into 0d41aeeeb0e95f9f6f143424e92a268c470f2a5a on alexvoronov:master.

This commit adds a lot of unnecessary blank lines. I'm not sure I agree with that formatting policy. Moving braces to new lines looks good though :)

Let us see if we can remove blank lines...
On Wed, Jun 15, 2016 at 4:44 PM, Alex Voronov<EMAIL_ADDRESS>wrote:

This commit adds a lot of unnecessary blank lines. I'm not sure I agree
with that formatting policy. Moving braces to new lines looks good though :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alexvoronov/geonetworking/pull/38#issuecomment-226191132,
or mute the thread
https://github.com/notifications/unsubscribe/AR09BGhsStAmGVM_Zfgs08yl6rn6vXt0ks5qMAG6gaJpZM4Iz2hn
.


Regarding obtaining permissions, you can consider LaunchedEffect

You can consider LaunchedEffect when obtaining permissions. Because it is a one-time use, you can avoid changing the ViewModel and causing continuous triggering of permission judgments.

@tenSunFree I dont know how I missed this. Thanks for reporting this.


Fix NPE when trying to build a DnsNameResolver with a null resolvedAddressTypes

Motivation:
It should be possible to build a DnsNameResolver with a null resolvedAddressTypes, defaulting then to DEFAULT_RESOLVE_ADDRESS_TYPES (see line 309).
Sadly, preferredAddressType is then called on line 377 with the original parameter instead of the instance attribute, causing an NPE when it's null.
Modification:
Call preferredAddressType with instance attribuet instead of constructor parameter.
Result:
No more NPE

@slandelle this looks good... Can you just also add a test ?

@normanmaurer done
I'm pretty swamped atm, but I'll try in the next days to stop using the deprecated constructor.

@slandelle thanks... two things to change tho. Also sorry that I was not more responsible for the dns stuff in the last weeks. Also super busy here :(

@netty-bot test this please

done

@slandelle thanks a lot!

Thanks for the review and merging!


Typo "@force_invalidate_snapshot"→"@force_invalidate_snapshot"

bold with escape characters
https://docs.microsoft.com/en-us/sql/relational-databases/system-stored-procedures/sp-changemergefilter-transact-sql?view=sql-server-ver15

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks.


improve error message when making syntax mistakes with ngFor directive

In the stack trace below, the iteration variable "hero" is used without the # prefix in the ngFor directive. The resulting error message doesn't clearly state that cause. Any chance of improving this error message ?
EXCEPTION: Template parse errors: Can't bind to 'ngFor' since it isn't a known native property ("
My Heroes

]*ngFor="hero of heroes" [class.selected]="hero === selectedHero"  (click)="onSelect(hero)">          "): AppComponent@3:16 Property binding ngFor not used by any directive on an embedded template ("
My Heroes

             [ERROR ->]
      "): AppComponent@3:12BrowserDomAdapter.logError @ angular2.dev.js:23514 angular2.dev.js:23514 STACKTRACE:BrowserDomAdapter.logError @ angular2.dev.js:23514

Please open an issue in https://github.com/angular/angular. Thanks!

you forgot to put hash #
try this
<li *ngFor="#hero of heroes"


Update README.md

You have a great README thus far. I would recommend referencing a code of conduct to make the repo welcoming.

How can I Improve the readme.md by referencing a code of conduct ?
Can you please elaborate on this ?

Sure! You can create a section with a Code of Conduct header, and below it describe that you are following the Contributor Covenant Code of Conduct. You can then provide context on why you did this, and why is it important to you and your community to have a code of conduct.

Fine ! Understood !
Thank You!


Add the option to specify an alternative helm binary

Helm may not be version compatible, how about option to overwrite command for switching other version client?
incompatible versions error sample is followings
Error: UPGRADE FAILED: incompatible versions client[v2.9.1] server[v2.8.2]

replaced 😋

Squashed commits while tweaking the commit message a bit


UX Design: Clean up Patient Information section on the patient treatment bar

Hey @Min2who - I don't have a directional UI, but this is breaking some consistency.. Vertical spacing, the format of the dropdown buttons..
Tailwind has some examples where the spacing looks right: https://tailwindui.com/components/application-ui/data-display/description-lists
If we can follow tailwind as a base I think it would put us in a good direction
If you have any thoughts on layout structure, feel free to take a stab at it
You can try out the demo here: https://demo.akello.io.. You will need to do the following

Sign up
Create a registry
Add a patient
Click on the patient to view the slide out drawer shown for this ticket

@Min2who -- we do have a discord server here. Feel free to join if you need any fast responses or need to discuss anything.

sure

i will solve this issue in a week or two
P.S:am occupied a bit with my exams lately

@Min2who - let me know if you would still like to work on this issue


Getting "frame too large error" with default configuration

Describe the bug
When I download and install the release build of go-dnscollector, I get errors and nothing is logged:
INFO: 2024/07/23 23:34:03.841482 worker - [tap] dnstap - conn #26 - new connection from <IP_ADDRESS>:59242 (localhost)
INFO: 2024/07/23 23:34:03.841568 worker - [tap] (conn #26) dnstap processor - enabled
INFO: 2024/07/23 23:34:03.842149 worker - [tap] (conn #26) dnstap processor - starting data collection
INFO: 2024/07/23 23:34:03.842253 worker - [tap] (conn #26) transformers applied: [normalize:qname-replace-nonprintable normalize:qname-lowercase]
INFO: 2024/07/23 23:34:03.842307 worker - [tap] (conn #26) dnstap processor - starting monitoring - refresh every 10s
ERROR: 2024/07/23 23:34:03.842582 worker - [tap] dnstap - conn #26 - stream initialization: frame too large error
ERROR: 2024/07/23 23:34:03.842603 worker - [tap] dnstap - conn #26 - framestream reader error: frame too large error
INFO: 2024/07/23 23:34:03.842653 worker - [tap] dnstap - (conn #26 - connection handler terminated
INFO: 2024/07/23 23:34:03.842712 worker - [tap] dnstap - conn #26 - cleanup the connection handler
INFO: 2024/07/23 23:34:03.842733 worker - [tap] (conn #26) dnstap processor - stopping monitor...
INFO: 2024/07/23 23:34:03.842738 worker - [tap] (conn #26) dnstap processor - monitor terminated
INFO: 2024/07/23 23:34:03.842741 worker - [tap] (conn #26) dnstap processor - stopping collect...
INFO: 2024/07/23 23:34:03.842749 worker - [tap] (conn #26) dnstap processor - collection terminated
INFO: 2024/07/23 23:34:03.842774 worker - [tap] dnstap - conn #26 - cleanup connection handler terminated

To Reproduce

Configure dnsdist to log to localhost port 6000:

Part of /etc/dnsdist/dnsdist.conf

-- Copied from https://github.com/dmachard/lua-dnsdist-config-examples/blob/main/logging_protobuf.lua
-- Needs https://github.com/dmachard/go-dnscollector to receive
-- init remote logger
protobuf_logging = newRemoteLogger("<IP_ADDRESS>:6000")
-- log all queries
addAction(AllRule(), RemoteLogAction(protobuf_logging, nil, {serverID="pdns server"}))
-- log all replies
addResponseAction(AllRule(), RemoteLogResponseAction(protobuf_logging, nil, true, {serverID="pdns server"}))
-- log all replies from cache
addCacheHitResponseAction(AllRule(), RemoteLogResponseAction(protobuf_logging, nil, true, {serverID="pdns server"}))
-- default rule
addAction( AllRule(), PoolAction("pdns"))

wget https://github.com/dmachard/go-dnscollector/releases/download/v1.0.0-beta1/go-dnscollector_1.0.0-beta1_linux_amd64.tar.gz

tar xzvf go-dnscollector_1.0.0-beta1_linux_amd64.tar.gz

./go-dnscollector

Expected behavior
It should "just work"?
Additional context

Version 1.0.0-beta1

Arghghgle.... wrong protocol selected!
Here's what I should have used in /etc/dnsdist/dnsdist.conf:
-- Copied from https://github.com/dmachard/lua-dnsdist-config-examples/blob/main/logging_dnstap.lua

-- listen on localhost
-- setLocal("<IP_ADDRESS>:53", {})
-- backend dns
------  newServer({address = "<IP_ADDRESS>:53", pool="default"})
-- init remote logger
tap_logging = newFrameStreamTcpLogger("<IP_ADDRESS>:6000")
-- log all queries
addAction(AllRule(), DnstapLogAction("dnsdist_server", tap_logging))
-- log all replies
addResponseAction(AllRule(), DnstapLogResponseAction("dnsdist_server", tap_logging))
-- log all replies from cache
addCacheHitResponseAction(AllRule(), DnstapLogResponseAction("dnsdist_server", tap_logging))
-- default rule
addAction( AllRule(), PoolAction("pdns"))

Ok, now I'm confused.
The logging_protobuf.lua example explicitly says
-- dnstap logging for dns traffic, can be used with the remote logger like https://github.com/dmachard/go-dnscollector

However the example doesn't work with the default configuration of go-dnscollector?
Next question: which protocol is better?

Next question: Assuming go-dnscollector can be configured to use either protocol, which protocol is better?

The default protocol, dnstap, is widely implemented across all DNS servers. On the other hand, the PowerDNS protocol (provided in  logging_protobuf.lua ) includes more metadata.
I prefer dnstap because its logs are not dependent on any specific DNS product.

You might want to include the corresponding dns server configurations as comments inside the example lua files; it'll help newbies like me.  😄

Semi-related note: https://dmachard.github.io/posts/0007-dnscollector-install-binary/ has example configuration using an older syntax.  May want to update that. 😄

You might want to include the corresponding dns server configurations as comments inside the example lua files; it'll help newbies like me. 😄

Maybe this can help
Enabling DNStap logging on most popular DNS servers

The problem I had was that DNSDist supports both Protobuf logging and DNSTAP (implemented with Protobuf), and DNSDist's syntax for both is very similar and easily confused.
It might help to include the correct DNSDist config with the matching DNSCollector config in the same file.
Or, if the two are so similar, then it might be worth it to combine the two and have the code auto-detect which it is.


Remove/Update Links

The links on the footer are currently linking to toladata.com, or MC for documentation.

Change documentation link to pint to the correct documentation location (Andrew to guide on this)
Update/remove TolaData link.
FAQ, Feedback and License links too.

@andrewtpham Please guide on this

Documentation for Activity-CE can be changed to: https://docs.arqaam.io/
Link to: http://www.toladata.org/
Link to new ticketing system (submit new ticket): https://arqaam.freshdesk.com/support/tickets/new

Or link to popup Widget:


Update event.html

Adjusted image placement on "Event" page to be positioned underneath the text.

@danielekeberg Could you look at these conflicts and decide with one of the two you want du keep?


Use a released version of mongoid-slug 5.0.0.

https://github.com/digitalplaywright/mongoid-slug/issues/190

Bump @aaw.

Sorry about that - pushed to rubygems as version 1.2.1

Oh thanks, apreciated! A release wasn't actually necessary since Gemfile is not used in the release itself. This was more like bookkeeping.


[Feature Suggestion] includeAll: <true/false> flag for test run creation including the whole suite or only the cases that are automated

The current solution creates a test run with all the test cases that exist in a TestRail test suite.
However, the suite contains test cases which are automated, and cases that are not yet automated; hence the test run metrics are including the untested cases as well.
It would be great to have a flag (such as includeAll: <true/false> ) to do the following:
if set to true, creates a test run with all the test cases within a test suite (as-is in the current state of the solution)
if set to false, gets a list of the test case IDs from automated suite {C123, C124, C125 ... } and creates a test run including only these test cases.
Here is a a script idea that parses the spec files and returns a list of case IDs, which can be used while creating the test run, maybe it could be useful:
caseIdExtractor.js
const fs = require('fs');
const path = require('path');

// Define the pattern to match test case IDs ending with either a colon or a space
const pattern = /C(\d+)(:|\s)/g;

// Function to extract test case IDs from files in a directory
function extractCaseIds(directoryPath) {
    return new Promise((resolve, reject) => {
        // List to hold all test case IDs
        let testCaseIds = [];

        // Read files from the directory
        fs.readdir(directoryPath, (err, files) => {
            if (err) {
                return reject('Error reading the directory: ' + err);
            }

            files.forEach(file => {
                const filePath = path.join(directoryPath, file);
                // Read the content of the file
                const content = fs.readFileSync(filePath, 'utf8');
                // Find all matches of the pattern
                let matches;
                while ((matches = pattern.exec(content)) !== null) {
                    // Convert matches to integers and add to the list
                    testCaseIds.push(parseInt(matches[1], 10));
                }
            });

            // Remove duplicates and sort the list
            testCaseIds = [...new Set(testCaseIds)].sort((a, b) => a - b);

            resolve(testCaseIds);
        });
    });
}

module.exports = extractCaseIds;

I execute it with the path to my spec files, and it prints a list of the case ids:

[ 2904, 3522, 3523, 9605, 10426 ]

@serhatgemici I tried your repo with the changes you mentioned above but but getting an error as 'illegal operation on a directory' and unable to get this to work.

@spoorthy89  you need to add TESTS_DIRECTORY into your config file and set it to the directory of your test files.
it still might or might not work, I'm using playwright-bdd, so my changes in my branch are supposed to extract case IDs from cucumber files (I haven't tried it with spec files), furthermore I customised my branch for my case where I wanted to make it create one single test run for parallel sharded runs on ci instead of creating multiple runs for each.

I was able to get it to work. Thank you @serhatgemici
Yes, this feature is really helpful.


import problem

Traceback (most recent call last):
File "", line 1, in
File "D:\annaconde\envs\tensorflow\lib\site-packages\openTSNE_init_.py", line 3, in
from .tsne import TSNE, TSNEEmbedding, PartialTSNEEmbedding, OptimizationInterrupt
File "D:\annaconde\envs\tensorflow\lib\site-packages\openTSNE\tsne.py", line 744
raise ValueError(f"Unrecognized initialization scheme {initialization}.")
^
SyntaxError: invalid syntax

Please note that you need Python 3.6 or higher to use openTSNE.


ATS 8.0.3 - abort in migrateToCurrentThread

It is crashing on this line:
   UnixNetVConnection *netvc = static_cast<UnixNetVConnection *>(netProcessor.allocate_vc(t));
Program received signal SIGABRT, Aborted.
[Switching to Thread 0x2aaab4107700 (LWP 32522)]
0x00002aaaadd121d7 in raise () from /lib64/libc.so.6
(gdb) bt full
#0  0x00002aaaadd121d7 in raise () from /lib64/libc.so.6
No symbol table info available.
#1  0x00002aaaadd138c8 in abort () from /lib64/libc.so.6
No symbol table info available.
#2  0x00002aaaad51e9b5 in __gnu_cxx::__verbose_terminate_handler() () from /lib64/libstdc++.so.6
No symbol table info available.
#3  0x00002aaaad51c926 in ?? () from /lib64/libstdc++.so.6
No symbol table info available.
#4  0x00002aaaad51c953 in std::terminate() () from /lib64/libstdc++.so.6
No symbol table info available.
#5  0x00002aaaad51d4bf in __cxa_pure_virtual () from /lib64/libstdc++.so.6
No symbol table info available.
#6  0x00000000006e0d00 in UnixNetVConnection::migrateToCurrentThread(Continuation*, EThread*) () at UnixNetVConnection.cc:1451
        netvc = <optimized out>
        client_nh = <optimized out>
        lock = {m = {m_ptr = 0x2aaaaef2ee50}, locked_p = true}
        lock_src = {m = {m_ptr = 0x2aaaaef2ed60}, lock_acquired = false}
        sslvc = 0x0
        save_ssl = <optimized out>
        ret_vc = <optimized out>
#7  0x0000000000522576 in HttpSessionManager::acquire_session(Continuation*, sockaddr const*, char const*, ProxyClientTransaction*, HttpSM*) () at HttpSessionManager.cc:332
        new_vc = <optimized out>
        server_vc = 0x2aaee231c6d0
        ethread = 0x2aaab141ac40
        pool_mutex = <optimized out>
        lock = {m = {m_ptr = 0x2aaaaef2fe90}, lock_acquired = true}
        to_return = 0x2aaab6f7b340
        match_style = <optimized out>
        hostname_hash = {b = {7978137750836667128,<PHONE_NUMBER>417706987}, u64 = {7978137750836667128,<PHONE_NUMBER>417706987}, u32 = {2948653816,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>},
          u8 = "\370\342\300\257\003\n\270n\353W\021\326\021\224\nr"}
        retval = HSM_NOT_FOUND
        __FUNCTION__ = "acquire_session"
#8  0x0000000000510f14 in HttpSM::do_http_server_open(bool) () at HttpSM.cc:4767

closing for now


GLBC - Issues about effectively replace / updating a certificate

I've successfully managed to setup an ingress with a certificate. However, I have not yet found a way to reliably replace the certificate on the ingress.
The documentation at:

https://github.com/kubernetes/contrib/tree/master/ingress/controllers/gce

does not mention updating or replacing certificates. I would suggest that a section is added to cover this.
I expected that when I replace the cert secret, and applied an replacement to the ingress object with a minor change (in example a route change), it would fetch the updated certs. This did not seem to happen.
Deleting the ingress, and recreating it did partly work, but creates severe downtime. I think that when creating the ingress with the same name, the old cert is cached/user.
Right now, I need to deploy a new ingress with a new ID in parallell, getting a new IP, and switch in DNS, before phasing out the old one. Very cumbersome.

Hmm, this should work. We have an e2e for it (https://github.com/kubernetes/kubernetes/blob/master/test/e2e/ingress_utils.go#L108). Are you running in GCE or GKE? What version Kubernetes?
I just tried and it does what I expect. Upon apply, you should see a second ssl cert, i.e:
$ gcloud compute ssl-certificates list
NAME                                                       CREATION_TIMESTAMP
k8s-ssl-default-echomap--d1834d65b5c525fd                  2016-10-21T15:23:56.041-07:00
$ kubectl apply -f /tmp/foo2.yaml

... wait a bit ...

$ gcloud compute ssl-certificates list
NAME                                                       CREATION_TIMESTAMP
k8s-ssl-1-default-echomap--d1834d65b5c525fd                2016-10-21T14:57:55.856-07:00

And on re-apply you should see it go back to the original name.
This is because edits to the resource arean't allowed, and we can't delete the cert while it's in use, so we need to create a side-cert (-1), and atomically swap it before deleting the old one, then repeat every time the secret changes.
There will be some delay in noticing the modified secret, because we're polling on secrets not watching, but it should work out. If you can give me repro steps I'll debug.

I have similar issues using Kubernetes 1.4.3. Unfortunately I am unable to upgrade the cluster at the moment because of the downtime, so if this has been fixed in a recent version I apologize beforehand.
These are my yaml's:
secrets.yaml
apiVersion: v1
kind: Secret
metadata:
  name: tls-crt
  namespace: production
type: Opaque
data:
  tls.crt: <chain crt>
  tls.key: <private key>

load-balancer.yaml
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: load-balancer
  namespace: production
  labels:
    tier: backend
  annotations:
    kubernetes.io/ingress.allow-http: "false"
spec:
  tls:
  - secretName: tls-crt
  backend:
    serviceName: api
    servicePort: 80
  rules:
  - host: api.<domain>.com
    http:
      paths:
      - path: /*
        backend:
          serviceName: api
          servicePort: 80
  - host: developer.<domain>.com
    http:
      paths:
      - path: /*
        backend:
          serviceName: developer
          servicePort: 80

When I update the base64 certificate string and update the secrets using kubectl update -f ./secrets.yaml, I get events with the following:
2016-10-31 10:44:00 +0100 CET   2016-10-30 20:06:52 +0100 CET   948       load-balancer   Ingress             Warning   GCE       {loadbalancer-controller }   googleapi: Error 409: The resource 'projects/<project_id>/global/sslCertificates/k8s-ssl-1-production-load-balancer--9aa517b40f2ca4ca' already exists, alreadyExists

Updating the load-balancer Ingress doesn't seem to have any effect either.

@svda I am experiencing the exact same problem, even today when I had to update.
In addition to update the secret itself, I have also tried to create a new secret, and patch the ingress to refer to the new secret: did not work... nothing happens.

@bprashanth Thanks for your hint wrt a certificate named k8s-ssl-1.... This probably was something laying around from when I did this operation 6 months ago. When I deleted this entry using google cloud console web page › compute › networking › load balancing › certificates, it suddenly worked for me to update the letsencrypt certificate without downtime! Awesome, thanks!

I had this same issue and removing the k8s-ssl-1* certificates manually from gcloud worked for me also. They were unused so it felt safe to nuke them. From there I watched the glbc.log log file on my kube master and could see it cleaning up and adding the desired certificates... Thanks!

Same error here, had a "...-1..." cert laying around, deleted it, and after a while (5-10 minutes) the new cert was swapped in. Kubernetes 1.5.4


Optimize the repetitive code

Screenshots
If applicable, add screenshots to help explain your problem.
To optimize the repetitive code in the file
pixelvibe/template/loader.html

I would like to work on this issue. Kindly assign it to me as a part of GSSoC.
Thank you!

We are not including this file in our webpage , its just added as an extra


[Bug]: fullscreen mode not showing lyrics

🔍 Have you checked that you are using an auto updating method or latest version of the extension?

[X] I have checked that I am on the latest version.

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Choose the extension for bug report.
Full Screen Mode
ℹ Environment / Computer Info
- Spotify version: <IP_ADDRESS>2.g1924b088
- Spicetify version: 2.19

📝 Description
when i put spotify in fullscreen mode it doesn't show the song lyrics. I tried uninstalling other extension and snippets that modify the lyrics tab but it's still not working..
📸 Screenshots

install lyrics+

already have it

You have to add lyrics-plus to the config, and then again run spicetify apply, it doesn't take automatically.
Check https://github.com/daksh2k/Spicetify-stuff/issues/26#issuecomment-1296229075 for steps.


Fixes documentation of v_scroll_speed and wrap_enabled

Fixed #25707
Modified the documentation regarding v_scroll_speed and wrap_enabled.

Thank you!

Cherry-picked for 3.1.1.


Moon phase widget - dark mode, custom colours, and small bugfix

Dark mode, custom colour for the moon, and fix a bug with acting on mylocation changes

Great, thanks!


chore(go.mod): extract replaces owned by scanner

Description
Assign replacements to @stackrox/scanner
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
go mod tidy
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#10409
#10408
#10407  👈
#10373 : 1 other dependent PR (#10390 )
#10375
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @janisz and the rest of your teammates on  Graphite

Merge activity

Mar 19, 4:57 PM GMT+1: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


Added support for pulling AST results.

Results now pull directly from AST Rest API.
Summary report now works
The results were supposed to be unmarshalled into structures in the sast-results library but those don't match the output from AST. I impelmented custom results structure in the CLI. This should be corrected in the future.

Scan submitted to Checkmarx

Checkmarx SAST - Scan Summary & Details
Cx-SAST Summary
Total of 7 vulnerabilities 0 High 0 Medium 7 Low 0 Info
Violation Summary
No policy violation found

Checkmarx SCA - Scan Summary & Details
Cx-SCA Summary
Total Packages Identified: 0
Scan Risk Score: 0.00
 0 High severity vulnerabilities
 0 Medium severity vulnerabilities
 0 Low severity vulnerabilities
View more details on Checkmarx UI
Cx-SCA vulnerability result overview
No package violation found


Error when exporting appleseed scene from project with packed textures

Repro case: tests.zip
Error:
Traceback (most recent call last):
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\render\__init__.py", line 138, in render
    self.__render_final(depsgraph)
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\render\__init__.py", line 244, in __render_final
    scene_translator.translate_scene()
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\translators\scene.py", line 247, in translate_scene
    textures_to_add)
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\translators\scene.py", line 1007, in __create_entities
    mat.create_entities(self.bl_scene)
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\translators\material.py", line 71, in create_entities
    self.__as_nodetree.create_entities(bl_scene)
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\translators\nodetree.py", line 62, in create_entities
    self.__create_shadergroup(bl_scene)
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\translators\nodetree.py", line 114, in __create_shadergroup
    AssetType.TEXTURE_ASSET, sub_texture)
  File "C:\Users\Franz\AppData\Roaming\Blender Foundation\Blender\2.82\scripts\addons\blenderseed\translators\assethandlers.py", line 135, in process_path
    shutil.copy(os.path.join(original_dir, filename), os.path.join(dest_dir, filename))
  File "C:\Program Files\Blender Foundation\Blender 2.82\2.82\python\lib\shutil.py", line 248, in copy
    copyfile(src, dst, follow_symlinks=follow_symlinks)
  File "C:\Program Files\Blender Foundation\Blender 2.82\2.82\python\lib\shutil.py", line 120, in copyfile
    with open(src, 'rb') as fsrc:
FileNotFoundError: [Errno 2] No such file or directory: 'D:\\cg\\appleseed\\repro cases\\drachenseele_normals\\unnamed.png'

Unpacking the project before exporting fixes the issue.

#452

Closing this one as an alert was added that stops the render when packed textures are detected. A more robust solution might be a good idea at some point but it'll take some work to implement.


satyam arrives

👷 Deploy Preview for rayko-research processing.

Name
Link

🔨 Latest commit
7173ab7bc93c9eab8b487209b228ddd2f854e1e1

🔍 Latest deploy log
https://app.netlify.com/sites/rayko-research/deploys/65dd952bb175990009b425e7


set binding implementation limits reuseability

Hey,
we are using 6.5.3 and wanted to implement some sort of conditionals by returning a NoopBinder in the case that the condition is false.
bind<Type> ()
conditional<Type> (predicate)

It' currently not possible to create a new TypeBinder because inSet is implemented as extension function.
inline fun <reified T: Any> Kodein.Builder.TypeBinder<T>.inSet() = InSet(generic())

This leads to a class cast exception if the type doesn't match kodein default implementation
We also tried to return some interfaces with a proper inSet definition, but it's currently also not so easy to create a ParameterizedTypeToken or type token in general, that would match the required key signature, resulting in a
Exception in thread "main" java.lang.IllegalStateException: No set binding to bind<Set<Type>>() with ? { ?

Do you have any hints how to create a proper parameterized token with TT?
We've tried with the following definition
bind () from setBinding<Type> ()

// key represented as -> bind<Set<out T>>() with ? { ? }

InSet (CompositeTypeToken (generic<Set<T>> (), TT (type)))

// key represented as -> bind<Set<T>>() with ? { ? }

Many thanks in advance

Hi. Kodein-DI 7.5 is out and the good news is that set bindings can now be detached from the TypeBinder.
example:
        val di = DI {
            bindSet<IPerson>()

            inSet<IPerson> { singleton { Person("Salomon") } }
            inSet<IPerson> { provider { Person("Romain") } }
        }

The same goes for argument set bindings.
About the TypeToken API, this has been entirely refactored with the Kodein-Type library.
Let me know if you still need help.


⚠ Bofrost.es has degraded performance

In ac97c99, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1718 ms

Resolved: Bofrost.es performance has improved in 3dfdcca after 2 hours, 43 minutes.


record user's telemetry preference

notify user by default about telemetry enabled, #1121

The formatting is broken
Successfully generated html-report to => /Users/zabilcm/Projects/jvowels/reports/html-report/index.html
Specifications:	1 executed	1 passed	0 failed	0 skipped
Scenarios:	2 executed	2 passed	0 failed	0 skipped

Total time taken: 138ms
Telemetry
	---------
	This installation of Gauge collects usage data in order to help us improve your experience.
	The data is anonymous and doesn't include command-line arguments.
	To turn this message off opt in or out by running 'gauge telemetry on' or 'gauge telemetry off'.

	Read more about Gauge telemetry at https://gauge.org/telemetry

Should be
Successfully generated html-report to => /Users/zabilcm/Projects/jvowels/reports/html-report/index.html
Specifications:	1 executed	1 passed	0 failed	0 skipped
Scenarios:	2 executed	2 passed	0 failed	0 skipped

Total time taken: 138ms

Telemetry
---------
This installation of Gauge collects usage data in order to help us improve your experience.
The data is anonymous and doesn't include command-line arguments.
To turn this message off, opt in or out by running 'gauge telemetry on' or 'gauge telemetry off'.

Read more about Gauge telemetry at https://gauge.org/telemetry

fixed via 0c046b7


Fix: Assert route names

This PR

[x] asserts the matched route names in all integration tests to prevent regressions

Related to #462.

Can this be merged? It's assigned to @Ocramius

Please rebase this onto develop branch.

:+1:


Use a timezone other than UTC for the node tests

In #4833, we set the TZ environment variable to UTC in tools/test-js-with-node.  While it's good to pin the timezone used for the unit tests, we probably will want to change that to something that is not UTC, so that any local-vs-UTC-time conversion issues manifest in the tests.
See the discussion in #4833 for more relevant details.

@zulipbot claim

Yes, i am still working on the issue

I tried to change the UTC value by an invalid value like "azeazae" and make a test by ./tools/test-js-with-node which return success
After that, i changed the valid value "UTC+2" and it has returned an error with the same test
why does it return success whereas the value is invalid ?
Thanks

@zulipbot i am always working on it

@zulipbot claim


refmt removes parens when it shouldn't

In index.re which uses ReactDOMRe to render a Reason-React top level component, I've added the following:
type foo = {
  sides: ref int
};

let bar = {sides: ref 8};

ReactDOMRe.renderToElementWithId <App sides=(!bar.sides) /> "root";

When saving, sides=(!bar.sides) is converted to sides=!bar.sides which results in a Error: 19: <UNKNOWN SYNTAX ERROR>.

Dupe of #869. I'm closing that one because this one's newer.

Thanks - for now, do baz.contents (which is the desugared form of !

This is fixed with the new syntax in master:
type foo = {sides: ref(int)};

let bar = {sides: ref(8)};

ReactDOMRe.renderToElementWithId(<App sides=bar.sides^ />, "root");

Is... that the new deference syntax? Woah...

Yes,  we got the  ! as in if (! predicate(arg1, arg2)) {... } back. Desugars to not.
I'm not sure however if that space is necessary, it's currently printed with an extra space after the !. It might have to do something with infix operators.

Yeah, though I didn't think about what the new .contents sugar would be. That's kinda weird

It should probably be changed before the next release

We've decided to keep that dereference sugar. Closing this. Fixed in master


ID-009

I really like how this works, but there is one big issue. Which is the JAV with ID of ID-009.
According to JAVLibrary there is 5 JAV's with that ID,
In future when this happens, maybe make the script ask which one is the right one?

Hey Yukkun,
I'm not sure if adding a prompt to match single videos would be feasible given how Javinizer currently runs with -Multi. I can take a look into it and see if I can maybe implement it in single sort mode.

Unfortunately with the direction I have taken Javinizer I don't think an interactive single-sort mode is in the works, at least not within the commandline. With the 2.x.x release, Javinizer now always run with multi-sort mode, but has improved functionality for sorting single files with direct URLs using the -Url parameter.


[release/5.0] Add webapp and worker to smoke-tests

Running worker application locally helped catch aspnet/common -> dotnet/extensions move. So lets add it to the smoke-tests as well.

cc @crummel

/azp run

Just want one more set of eyes on this and then I'd like to get it in.


incremental builds: add derivation override functions

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Intriguing!
Some thoughts:

buildOut: "out" is a little redundant
the intermediate files are more likely not to be reproducible than normal outputs
most of all: Please write tests

Intriguing!
Some thoughts:
* buildOut: "out" is a little redundant

* the intermediate files are more likely not to be reproducible than normal outputs

* most of all: Please write tests

Thanks for your input.
I have changed the names and added a test. Let me know If there is more to do for testing.
To your second concern:
I guess the intermediate files should be as reproducible as normal outputs, as they depends on the input and the compiler, also.
But I can not guarantee that they will be reproducible at any time.
I run the same derivation several times, and the derivation would not be rebuilt if nothing in the source tree changes.

This is pretty cool.
Interface wise i think it might be nicer to use and it seems like nothing speaks against doing it like this maybe?
incrementalDrv = pkgs.buildIncremental drv;

this could be done by implementing build-incremental.nix like this:
{ pkgs }:
let
  prepareIncrementalBuild = drv: ...;
  mkIncrementalBuild = drv: ...;

in

drv: previousArtifacts:

mkIncrementalBuild (prepareIncrementalBuild drv) previousArtifacts

The tests would look slightly different but would show the same thing.

This is pretty cool.
Interface wise i think it might be nicer to use and it seems like nothing speaks against doing it like this maybe?
incrementalDrv = pkgs.buildIncremental drv;

this could be done by implementing build-incremental.nix like this:
{ pkgs }:
let
  prepareIncrementalBuild = drv: ...;
  mkIncrementalBuild = drv: ...;

in

drv: previousArtifacts:

mkIncrementalBuild (prepareIncrementalBuild drv) previousArtifacts

The tests would look slightly different but would show the same thing.

You can not handle it that way, because there is no possibility to detect an earlier build automaticly, as nix is lacking the corresponding Support.

You can not handle it that way, because there is no possibility to detect an earlier build automaticly, as nix is lacking the corresponding Support.

Ok sorry for the noise. Now i get how it works.

What happens when a source file is removed in the new build? It seems unavoidable that the accompanying intermediate file will be present, which is a risk, but a decent build script will only glob for source files.
This needs a test case.

In the existing test case, could you check that unmodified files aren't recompiled?

What happens when a source file is removed in the new build? It seems unavoidable that the accompanying intermediate file will be present, which is a risk, but a decent build script will only glob for source files.
This needs a test case.
Yes, good Idea, I will add it

In the existing test case, could you check that unmodified files aren't recompiled?

Yes.

I've added the requested tests.

Does this require two copies of src? this doesn't seem to create genuine incremental builds, but require one "unaltered" source, that acts as a seed for the 2nd derivation. This can be problematic when you want to work on big projects like linux source code.

Does this require two copies of src?

Yes.

this doesn't seem to create genuine incremental builds

That is correct. "True" incremental builds are a cross-cutting concern that goes all the way down to the build tool used inside the sandbox. As such, Nix can not be solely responsible for managing the state, making such a solution fragile, unless the build tool can delegate the state management to recursive nix, which is, still, rather restrictive and an invasive requirement.
Incrementality can't be added in a transparent manner, so instead, this is an embodiment of an incremental build within the derivation paradigm, that makes the prior state explicit and does not provide any guarantees beyond the fact that the result is an incremental build from a specific version to another specific version.
@messemar Could you explain this in the Nixpkgs manual?

This can be problematic when you want to work on big projects like linux source code.

If you buy into Nix, you have to be ok with "wasting" some space for the benefits of reproducibility, atomicity, etc.
There's two kinds of space waste: the temporary and the permanent. The first just causes garbage collection to run more frequently, the latter should not be affected by this solution.
When you're rebuilding, say, linux you are already copying the src to the store for each new iteration (unless you only change the expression), so presumably the garbage produced isn't much of a problem.
I guess I'm skipping over some subtleties, but I think this explanation is already getting quite long.

@messemar Could you explain this in the Nixpkgs manual?

Yes I can, when I have some spare time again I will. As well as continuing the implementation, of course :D

Steps to un-draft: Add a section in the nixpkgs manual.

@roberth PTAL, I don't see any reasons apart from the ones I commented to not merge this. Is this fine by you?
Also cc @infinisil since you know a lot about things :)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixpkgs-support-for-incremental-haskell-builds/24115/2

I want to mention that I have a PR open against Nix which would make it possible to detect the earlier build automatically:
https://github.com/NixOS/nix/pull/7362
I also have a blog post explaining this in more detail:
https://www.haskellforall.com/2022/12/nixpkgs-support-for-incremental-haskell.html

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixpkgs-support-for-incremental-haskell-builds/24115/7

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixpkgs-support-for-incremental-haskell-builds/24115/8

@roberth I would love to see it merged! I've just decided to make it a standalone project, to make it available to others in some way.

This would be brilliant to have available as a tool. Long-running builds from scratch due to tiny source changes are a major headache at times, and to be able to use this kind of compromise when it makes sense would be a huge improvement in daily work. I‘m looking forward to see this go in!

@roberth @dasJ  Is there anything left, that I can do to get this over the finishing line?

There isn't

Ah there is: @infinisil is it okay to dismiss your review?

Ah there is: @infinisil is it okay to dismiss your review?

Oh yeah sorry missed this. It's okay. Generally this does seem a rather hacky, but it looks like it's good enough for a lot of use cases, so it's fine.
@messemar: Though it looks like https://github.com/NixOS/nixpkgs/pull/167670#discussion_r1052295282 wasn't fully addressed, and I can't find a comment explaining why.

rather hacky

I'm sure you think the same about it, but to be clear, the goal itself is hacky, as something non-hacky would have to integrate with something other than phases perhaps, in order for it to be able to give guarantees that can't be messed up by the user of this feature, but it's unclear to what that would be.
The implementation is not at all hacky in and of itself.

@infinisil and @roberth
I totally agree on the hackyness. But as checkpoints and incremental builds are not a first class concept for nix, it can not be enabled in a non hacky way, I think.
Regarding https://github.com/NixOS/nixpkgs/pull/167670#discussion_r1052295282 If we use dotglob wouldn't that mean that we also remove ´.´ and ´..´ as well ? Does not sound like a good Idea IMHO. But yes, we might miss dotfiles then.

@messemar You do have dotglob enabled in the PR. I was thinking of the other point in that comment that allows avoiding the extglob for rm -r !("sourceDifference.patch").

@infinisil Ah,  now I get it.  Is it a nit, or should I fix it in a follow up PR ?

@messemar I'd classify it as a nice-to-have for correctness. Feel free to make a follow-up PR, but not necessarily :)

Would it make sense to mention this in the 2023 recap?

@asymmetric It needs an explicit starting state, and it only works for typical mkDerivation based builds. That should be made very clear. I would also like to see it used in practice somewhere.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/idea-mkderivation-auto-generate-unwrapped-package-right-after-build/37901/3

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/idea-mkderivation-auto-generate-unwrapped-package-right-after-build/37901/1

I really, really like this idea and would like to use it in practice! However, there are some minor inconsistencies across the docs; e.g. mkCheckpointBuild is mentioned in the doc:
https://github.com/NixOS/nixpkgs/blob/d6b045fcc3d2eb036e16b04b1127f1457eb242d7/doc/build-helpers/special/checkpoint-build.section.md#L34-L36
But the actual command is mkCheckpointedBuild with an ed.
In fact, all other functions in this PR are in the form of *CheckpointBuild* without the ed, so I think we might as well trade the mkCheckpointedBuild name in favor of mkCheckpointBuild for an uniformed look (and link mkCheckpointedBuild to mkCheckpointBuild with a lib.warn "deprecated", although currently there is no usage of this in nixpkgs). Is that okay? If so, I can make a PR!

In fact, all other functions in this PR are in the form of CheckpointBuild without the ed, so I think we might as well trade the mkCheckpointedBuild name in favor of mkCheckpointBuild for a uniformed look (and link mkCheckpointedBuild to mkCheckpointBuild with a lib.warn "deprecated", although currently there is no usage of this in nixpkgs). Is that okay? If so, I can make a PR!

@bryango From my perspective, the change is reasonable and I like it.


measurement adjustment from measurement window

user should be able to adjust start and end point from the measurement window


Baremetal install hangs with a yaml unmarshal error

When doing a bare metal install of v0.3.0-rc4, the reboot hangs when unmarshalling cloud-config-processed.yml
The line points to the ssh public key.  In the .yml file, the line is split with the user@host part on the next line.  This happens for all the .yml files.  Editing these files, allows the boot to continue to completion.
The output of sudo rancherctl config export -p has the same formatting problem:
    rsa-pub: ssh-rsa AAAA...
      root@rancher

How did you do the installation?  What does your cloud-config look like that you passed to rancheros-install?  Did it also have a newline in it?  Typically the cloud-config-processed doesn't have rsa-pub in it.

I used rancheros-install with a cloud-config.yml file that looked like the following.  The ssh_authorized_keys value was all on one line:
#cloud-config

ssh_authorized_keys:
  - ssh-dss AAAA... noon@host

# Anything you can put in the rancher.yml
rancher:
  network:
    dns:
      nameservers:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
    interfaces:
      eth0:
        address: <IP_ADDRESS>/24
        gateway: <IP_ADDRESS>
      eth1:
        address: <IP_ADDRESS>/16
        gateway: <IP_ADDRESS>
      lo:
        address: <IP_ADDRESS>/8

The rss-pub entry is in rancher-private.yml

Thanks, I'll try to reproduce on my side

I tried this and can't reproduce the error.  Here is what I did.  I created an empty HD and booted the RancherOS v0.3.0 ISO.  I logged in with rancher/rancher and created /home/rancher/cc.yml with the below contents
#cloud-config

ssh_authorized_keys:
  - ssh-dss AAAA... noon@host

# Anything you can put in the rancher.yml
rancher:
  network:
    dns:
      nameservers:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
    interfaces:
      eth0:
        address: <IP_ADDRESS>/24
        gateway: <IP_ADDRESS>
      eth1:
        address: <IP_ADDRESS>/16
        gateway: <IP_ADDRESS>
      lo:
        address: <IP_ADDRESS>/8

I then ran sudo rancheros-install -d /dev/sda -c /home/rancher/cc.yml.  After reboot it worked.
I tried sudo rancherctl config export -p and validated that it does indeed send a new line for rsa-pub.  This appears to be a valid yml syntax for strings in the "flow style".  I tried sudo rancherctl config export -p | sudo rancherctl config import and this also worked.

I guess the cloud-config yam syntax was a red herring.  I did a clean install with v0.3.0 on a different system and it installed cleanly.
Thanks for your help (and for rancher!)


Display the error why a rails server is not starting during kamal setup

Hi Kamal team,
When I ran kamal setup, I got the message saying the healthcheck has failed.
 ERROR {
  "Status": "unhealthy",
  "FailingStreak": 27,
  "Log": [
    {
      "Start": "2024-03-25T06:10:57.057607108Z",
      "End": "2024-03-25T06:10:57.150344418Z",
      "ExitCode": 1,
      "Output": "  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current\n                                 Dload  Upload   Total   Spent    Left  Speed\n\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\r  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0\ncurl: (7) Failed to connect to localhost port 3000 after 3 ms: Couldn't connect to server\n"
    },

This is too generic and unactionable. A more helpful error should not only show that health check has failed, but why the health check failed by displaying the error message that would've printed, if you ran ./bin/rails s locally.
I was able to dubug what the real issue is by running the command to manually start the docker app image like:
docker run --detach --name healthcheck-kamal-rails-pg-tutorial-ccc7f8046c5feaa21e3cd52b9c69fed49dbd413c_uncommitted_ba1a020121f126ac --publish 3999:3000 --label service=healthcheck-kamal-rails-pg-tutorial -e KAMAL_CONTAINER_NAME="healthcheck-kamal-rails-pg-tutorial" --env-file .kamal/env/roles/kamal-rails-pg-tutorial-web.env azuretutorials.azurecr.io/serv/kamal-rails-pg-tutorial:ccc7f8046c5feaa21e3cd52b9c69fed49dbd413c_uncommitted_ba1a020121f126ac

@serv - before the failed healthcheck message you've posted, you should have the output from the container logs. See https://github.com/basecamp/kamal/blob/67a86e1068d312b80726890805328d0958609534/lib/kamal/cli/healthcheck.rb#L12.
Did you see anything there?


file

Hey developer,
I just wanted to know that how to generate the filedetails.json in very detail.
I have googled many times but i didn't get the answer how to generate this file.
And wanted to thank you in advance for this wonderful project;

Or can you just give me the link of any tutorial or video step by step how can I generate that file.
I'll be very thankful for that.
And thanks again which is always appreciable.

Or can you just give me the link of any tutorial or video step by step how can I generate that file.
I'll be very thankful for that.
And thanks again which is always appreciable.

Hi i am using this site https://jsoneditoronline.org/

https://www.youtube.com/watch?v=mjwy9YHMFhU&t=12s


Bugfixes, verbosity and error handling

see commit messages and the changes themselves
I am not at all experienced with click. It seems that it is not possible to mark a parameter as required, so I did it on my own.

Nice stuff in here. For some reason I did not get a notification about this PR. There has been quite some changes from my side as well. Could you go over the commits again, please?
Specific things to mention, are that click actually supports required options:
https://click.palletsprojects.com/en/7.x/options/#basic-value-options - I've added these already.
I did some changes to setup.py.


HTTP-CoAP proxy

Expose HTTP Proxy Endpoint at root /
if Request starts with coap/coaps
if not enabled/proxy not allowed, return 5.01
if timeout, return 5.04
if response not understood, return 5.02
cross check allowed coap accept headers allowed - returns 406 not acceptable
Methods

OPTIONS
return 5.01
TRACEw
return 5.01
GET
if Success return 2.00
response payload == CoAP Representation resource, Content-Type and Content-Encoding header must be set

Include to CoAP Request

Accept, Conditional GET
If Accept, validate Accept is valid, else return 4.06

HEAD

Like GET but no message body response

POST

If the action performed by the POST method does not result in a resource that can be identified by a URI, a 200 (OK) or 204 (No Content) response MUST be returned to the client.
If a resource has been created on the origin server, a 201 (Created) response MUST be returned.
If CoAP responds with Location-, return to HTTP the Location- Headers

PUT, DELETE
CONNECT
return 5.01

implemented f6d5994a9fe39fcc7366e86c7f14fa42c6d2a4dc


Working directory is now optional

Related to #1915

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@DorSoluto your changes have been merged, thanks for your contribution 👍


Have Stream Channel from native code to flutter

Use case
Flutter has method channels and event channels through great,they do have their limitations,for instance when the native code is listening to data from a web socket or stream from the network the native code has to invoke a event channel to send the data to the flutter view.

Proposal
What if there could be a full duplex communication between the flutter code and the native code.
With reactive programming when an object is sunk into the stream the listener whether native or
flutter could get instant notification.
This would reduce the need to invoke function, rather listen to a stream then react to the stream irrespective of the platform.

after some research .
I came  across to this article.
https://medium.com/@svenasse/flutter-event-channels-89623ce6c017


VS2017: completion bug on provided members with . in them

I'm facing some issues with completion and colorization in VS2017 which didn't exist in previous release.
completion bug  of TP members
When I pick members which are to be escaped with ``, it doesn't escape them when committing the completion
VS2017:

VS2015:

(the popup don't show in capture)
weird completion behaviour in TP arguments
trying to complete with CTRL+Space at the .Text with carret anyplace after T:

it completes with

This is, I gather, a hidden TP generated member
colorization of TP arguments
The colorization seems off when a type argument is using the `` names:
VS2017:

VS2015:

Might be related to #2585
attaching a minimal environment (you'll need to adjust connection string to a SQL Server available to you)
tpcompletion.zip

I've completely forgot, but it may be the case that I fixed some of these bugs in https://github.com/Microsoft/visualfsharp/pull/2235. Maybe you could resurrect it and check?

@vasily-kirichenko I see, I'll try to pull that and test it with my case, thanks!

memo: @cloudRoutine mentions I should look at CompletionProvider.fs to troubleshoot that issue
    override this.GetChangeAsync(_, item, _, _) : Task<CompletionChange> =
        match item.Properties.TryGetValue IsExtensionMemberPropName with
        | true, _ -> ()  // do not add extension members to the MRU list
        | _ ->
            match item.Properties.TryGetValue FullNamePropName with
            | true, fullName ->
                match mruItems.TryGetValue fullName with
                | true, hints -> mruItems.[fullName] <- hints + 1
                | _ -> mruItems.[fullName] <- 1
            | _ -> ()

        let nameInCode =
            match item.Properties.TryGetValue NameInCodePropName with
            | true, x -> x
            | _ -> item.DisplayText

        Task.FromResult(CompletionChange.Create(new TextChange(item.Span, nameInCode)))

@smoothdeveloper that code just replaces completion item name with nameInCode, which we stored into properties earlier. You should look at how we form it here https://github.com/Microsoft/visualfsharp/blob/master/src/fsharp/vs/ServiceDeclarations.fs#L1501

@cloudRoutine / @vasily-kirichenko thanks for the pointers, the first bug should be fixed in #2649 (and hopefully no regression).
The second one looks a bit esoteric, hopefully it is also happening around same place.


Project Check In #1

I have implemented these two screens following the apple developer guide. I am currently working on a "feed" that will replace the list format of the first frame and instead will cycle through the topics in the format shown in the second frame.
I need to figure out how to change the JSON of the topic once the user clicks either like or dislike. I also need to figure out how to implement functions... I am planning on revisiting the MVVM structure from the memorize app I built following the Stanford tutorial. I am having a hard time figuring out how pages connect. With web design, HTML is the format, CSS is the design, and JavaScript holds the functions. With swift, I have no idea where things are supposed to go and how to connect them together. I also cannot seem to find a simple break down of this on the internet (we should go over this in class wink wink)
Checklist:

make like and dislike button work (change the "isLiked" and "isDisliked" variables in JSON)
connect like and dislike buttons so when like is pressed unlike is unchecked (can you have one button with 2 icons?)
figure out how to make the feed, I think I could use a ForEach (I tried this and it failed but I will try again)
figure out how the user will "scroll" through the feed (buttons? swiping?)
make it cute because she is the trenches right now
add more JSON data (maybe categories the polls "animals", "food", etc.)
add a daily poll (DP) feature
add a user profile feature showing their likes and dislikes
add a navigation bar to get between the features (if I can implement the other features)

I am really excited about this app but feel really lost on the general process of making a swift app. I am slowly chipping away at it as my eye balls melt into my computer screen, but if you know of any resources that can explain swift logic in a simple way please share it with me. I need someone to draw out on a white board how the different .swiftui and .swift pages connect.
SOS ZEF
xoxo
nancyyoder :)

Also side note, I made a new app repo because things got really scary in the old one. The new one is MAD-Polls, and I updated this on my config.yml so you should be able to find it but if not let me know :)


fixFix drop stats fail silently bug.

Drop stats use IN predicate to filter the column stats to delete. The default length of IN predicate is 1024, drop table stats with more than 1024 columns may fail.
This pr is to split the delete sql based on the IN predicate length.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Refresh token fails

The adapter stopped working for me, see log below. Interesting to note: It says it'll retry in 5m, but next log entry is only three seconds later?!? This probably triggers rate-limit!
homeconnect.0 | 2022-02-06 22:31:23.517 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:31:23.517 | error | Error Refresh-Token: 429 undefined
homeconnect.0 | 2022-02-06 22:31:20.371 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:31:20.370 | error | Error Refresh-Token: 429 undefined
homeconnect.0 | 2022-02-06 22:31:17.977 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:31:17.977 | error | Error Refresh-Token: 429 undefined
homeconnect.0 | 2022-02-06 22:31:17.663 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:31:17.663 | error | Error Refresh-Token: 429 undefined
homeconnect.0 | 2022-02-06 22:31:15.380 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:31:15.379 | error | Error Refresh-Token: 429 undefined

Adapter restart seems to at least "heal" the 5m retry timespan:
homeconnect.0 | 2022-02-06 22:36:30.031 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:36:30.031 | error | Error Refresh-Token: 429 undefined
homeconnect.0 | 2022-02-06 22:31:37.914 | info | BOSCH-SBV68TX06E-XXXXXXX: There is no program active
homeconnect.0 | 2022-02-06 22:31:34.542 | info | BOSCH-SBV68TX06E-XXXXXXX: There is no program active
homeconnect.0 | 2022-02-06 22:31:29.932 | warn | Retry Refresh Token in 5min
homeconnect.0 | 2022-02-06 22:31:29.931 | error | Error Refresh-Token: 429 undefined
homeconnect.0 | 2022-02-06 22:31:29.701 | info | starting. Version 0.0.37 in /opt/iobroker/node_modules/iobroker.homeconnect, node: v14.18.2, js-controller: 3.3.22
homeconnect.0 | 2022-02-06 22:31:29.692 | info | Plugin sentry Sentry Plugin disabled for this process because sending of statistic data is disabled for the system
homeconnect.0 | 2022-02-06 22:31:25.831 | info | Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason
homeconnect.0 | 2022-02-06 22:31:25.830 | info | terminating
homeconnect.0 | 2022-02-06 22:31:25.828 | info | cleaned everything up...
homeconnect.0 | 2022-02-06 22:31:25.828 | info | Got terminate signal TERMINATE_YOURSELF

Had the same issue yesterday. Now it work probably again. Maybe create a new app token and restart auth process as documented.

If I hadn't done that already just a few days ago, I'd have said nothing..

fixed with 1.0.0


Update serialization and deserialization of enum types

The enum type is always serialized and deserialized to a single type (the Python ‘i’ type):
https://github.com/fprime-community/fprime-tools/blob/a65f0140e0e1f0ba40522767839059a78eaa3067/src/fprime/common/models/serialize/enum_type.py#L72-L100
We need to revise this code as follows:

Store the integer width in the Python enum (fprime-tools).
Represent the integer width in the AppDictionary.xml (nasa/fprime).
Get the integer width from the AppDictionary.xml when creating the Python enum (fprime-gds).
Use the stored integer width to serialize and deserialize the enum (fprime-tools).

Logging the issue here, but three repos are involved (fprime-tools, fprime-gds, and fprime).

@bocchino Why do we need to store the width of an enum type? Are we trying to support more than 255 members in an enum?

Moved to: https://github.com/nasa/fprime/issues/2229


Grand Crystal Tree not finishing plant

Yesterday the Grand Crystal Tree wasn't ticking off for the Farm Contract.
Today, I've tried to send the trip to harvest and replant twice, both trips did not send a return message, and after checking the patches, did not harvest or replant at all.
EDIT: Also still using seeds.

Same happened to me just now. Tried twice and both trips failed to return properly.

Fixed.
Please run =b first, then you will be able to Harvest your patch.


server: don't export APIs if feature is not enabled in any zones

This change will ensure that B&R APIs are not exported if the feature is not enabled in any of the zones.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1534

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1994)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 7573 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4202-t1994-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_backup_recovery_dummy.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Smoke tests completed. 28 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestDummyBackupAndRecovery>:setup
Error
0.00
test_backup_recovery_dummy.py

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1536

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@davidjumani please review this, thnx

Trillian test result (tid-2008)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 49880 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4202-t2008-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_backup_recovery_dummy.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 81 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestDummyBackupAndRecovery>:setup
Error
0.00
test_backup_recovery_dummy.py

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Error
470.84
test_vpc_redundant.py

test_04_rvpc_network_garbage_collector_nics
Error
3853.05
test_vpc_redundant.py

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔debian. JID-1539

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Packaging result: ✔centos7 ✔debian. JID-1547

Tested on KVM env, it skips tests for dummy provider now:
Import provider backup offering from Dummy Backup and Recovery Provider ... SKIP: Skipping test cases which must only run for Simulator
Test VM backup lifecycle ... SKIP: Skipping test cases which must only run for Simulator

----------------------------------------------------------------------
Ran 2 tests in 0.124s

OK (SKIP=2)


Link in "About" is broken

This link does not work anymore: https://play.google.com/store/apps/details?id=com.veewo.a1024

https://play.google.com/store/apps/details?id=com.androbaby.game2048&hl=en&gl=US

It's been like this for over 7 years now. Nothing we can do about it. Except...
@gabrielecirulli, can you change the 1024 link to the archive?
re #376, #354

Sure, done. Sorry for not changing this before!


Streamline highly available MySQL creation

What problem are you facing?
I'm working on GCE and trying to create a highly available MySQL on CloudSQL. I'm aware that CloudSQL simply doesn't make it very easy to do this, but I think that Crossplane could really shine by making this user experience a lot more automated. This was discussed on Slack: https://crossplane.slack.com/archives/CEG3T90A1/p1576754003006800
How could Crossplane help solve your problem?
Currently, the suggestion on Slack to create such a setup was:

I haven't done that exactly. But I imagine the following should work: A CloudSQLInstanceClass with properties you want to have on master, create a MySQLInstance that refers to that and let it create a master CloudSQLInstance resource. Then create another CloudSQLInstance CloudSQLInstanceClass that refer to that CloudSQLInstance  resource's name as master.
I know this isn't ideal. There is some work going on making this easier via introducing custom claim definitions that would enable one-to-many claim<>managed resource relation but the suggestion above should work
Oh another thing, if you don't want to do this serially, you can choose a name for your master like my-master-instance  and use it in replica CloudSQLInstanceClass as master name. Then before you create MySQLInstance of master, add the following as annotation to the MySQLInstance  resource "crossplane.io/external-name": "my-master-instance" . This will let you override the default naming schema. So, you can create master and replicas in parallel instead of waiting for master to be created, get name and use it in other resource class

However, if the MySQLInstance resource would have a "ha" Setting which would take care of this instead, the user experience would be a lot better.
Several suggestions were made on how to implement this:

I think the current thinking on how this will be handled is that we could have a claim that maps to multiple resources. This means you could use MySQLInstance or create your own like HAMySQLInstance that either claims an RDSInstanceClass with HA enabled or claims a set of of GCP classes that work together to provide an HA Cloud SQL instance :slightly_smiling_face:

And

I was actually thinking of a different solution where a MySQLInstance with a ha option set would claim a HACloudSQLInstanceSet or something, but I'm not a dev :wink:

I also ran into this and researched the topic a bit and as it turns out, there is a new HA type for MySQL which works similarly to what Postgres already has. I could not find anything in the documentation but the GCP console UI actually lets you create REGIONAL MySQL instances so I went looking in the logs what that API call looks like. It is exactly the same as with Postgres, you just need to set availabilityType: REGIONAL and make sure you have binlogs enabled and then it will create a HA instance for MySQL. As this field won't be filtered out it already works with the current crossplane gcp-stack:
apiVersion: database.gcp.crossplane.io/v1beta1
kind: CloudSQLInstance
metadata:
  name: ha-crossplane-1
spec:
  forProvider:
    databaseVersion: MYSQL_5_7
    region: europe-west6
    settings:
      availabilityType: REGIONAL
      tier: db-n1-standard-1
      dataDiskType: PD_SSD
      backupConfiguration:
        binaryLogEnabled: true
        enabled: true
        startTime: "00:00"
  reclaimPolicy: Delete
  providerRef:
    name: gcp-provider